-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[fix](stacktrace) Add conf enable_stacktrace
#37713
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 40024 ms
|
TPC-DS: Total hot run time: 173055 ms
|
ClickBench: Total hot run time: 30.63 s
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 40067 ms
|
TPC-DS: Total hot run time: 173872 ms
|
ClickBench: Total hot run time: 30.84 s
|
6d08a61
to
1a47d71
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39977 ms
|
run buildall |
run buildall |
TPC-H: Total hot run time: 39935 ms
|
TPC-DS: Total hot run time: 172944 ms
|
ClickBench: Total hot run time: 31.81 s
|
run buildall |
TPC-H: Total hot run time: 40053 ms
|
TPC-DS: Total hot run time: 173615 ms
|
ClickBench: Total hot run time: 30.88 s
|
run buildall |
1 similar comment
run buildall |
TPC-H: Total hot run time: 40145 ms
|
TPC-DS: Total hot run time: 174158 ms
|
ClickBench: Total hot run time: 30.76 s
|
PR approved by at least one committer and no changes requested. |
#ifndef ENABLE_STACKTRACE | ||
LOG(INFO) << stack_trace_st.to_string(); | ||
#endif | ||
if (!doris::config::enable_stacktrace) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be if (doris::config::enable_stacktrace)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YES
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it is if (!doris::config::enable_stacktrace)
u can see before it was #ifndef
ENABLE_STACKTRACE
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
执行到这里说明触发了概率性AllocateFailed,为了确保这里有且仅有一个堆栈,加了一个check,如果编译代码的时候不打堆栈,那这里手动获取堆栈打印到日志里。
`enable_stacktrace` if false, turn off all stacktrace
`enable_stacktrace` if false, turn off all stacktrace
enable_stacktrace
if false, turn off all stacktrace