Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CUBRIDQA-1240] Added the showTrace option to SystemModel in CTP to refrain from converting numeric values to '?' for trace statistics when set to true #690

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

junsklee
Copy link
Contributor

@junsklee junsklee commented Nov 27, 2024

Refer to: http://jira.cubrid.org/browse/CUBRIDQA-1240

SHOW TRACE; 를 CTP에서 사용했을때 출력되는 trace statistics에 관하여 showTrace 옵션을 도입할 것을 제안합니다. 본 옵션은 사용자가 트레이스 통계에서 숫자 값을 유지하거나 '?'로 변환할 수 있는 설정을 제공합니다.

해당 옵션은 다음과 같이 동작합니다:

In CTP/sql/configuration/System.xml,

  • showTrace = false (기본값)

    <showTrace>false</showTrace>

    false(기본값)로 설정된 경우, 기존 동작과 동일하게 SHOW TRACE 출력의 숫자 값이 자동으로 '?'로 변환됩니다.

  • showTrace = true

    <showTrace>true</showTrace>

    true로 설정된 경우, SHOW TRACE 출력에 변환되지 않은 기존의 숫자 값이 표시됩니다.

Notes

  • CTP/sql/configuration/System.xml 파일에 showTrace 옵션이 추가되면 regression의 장비들은 매번 CTP/common/script/upgrade.sh 파일을 실행하기에 System.xml 파일도 CTP 엔진과 함께 업데이트 됩니다. 이러한 이유로 System.xml과 CTP 엔진이랑 버전이 맞지 않는 경우는 드물며, 이 feature를 로컬 장비에서 사용하시려면 본 PR이 머지가 된 이후에 upgrade.sh 파일을 실행하시고 System.xml을 수정하시길 바랍니다.
  • JDBC only (cci 미지원)

…from converting numeric values to '?' for show trace when set to true
@junsklee junsklee marked this pull request as ready for review November 27, 2024 05:57
@junsklee junsklee marked this pull request as draft November 27, 2024 06:03
@junsklee junsklee marked this pull request as ready for review December 3, 2024 05:03
@junsklee junsklee requested a review from tw-kang December 5, 2024 07:01
@junsklee junsklee merged commit 45ff9b5 into CUBRID:develop Dec 12, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants