-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[improve](stream-load) add observability on receiving HTTP request #30432
Conversation
dff7b50
to
882d93d
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39270 ms
|
TPC-DS: Total hot run time: 176242 ms
|
ClickBench: Total hot run time: 31.01 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
882d93d
to
c81ad8d
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38966 ms
|
TPC-DS: Total hot run time: 175860 ms
|
ClickBench: Total hot run time: 30.64 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
c81ad8d
to
cbcb699
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 37404 ms
|
TPC-DS: Total hot run time: 173064 ms
|
ClickBench: Total hot run time: 31.15 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
97f596a
to
caabe77
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 37733 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 172596 ms
|
ClickBench: Total hot run time: 30.63 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
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
5b4e784
to
424a208
Compare
424a208
to
3c2af3a
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 37390 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 172986 ms
|
ClickBench: Total hot run time: 30.31 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
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. |
Proposed changes
We meet occasional slow stream-load, ReadDataTimeMs is very fast but WriteDataTimeMs is too long, according to some logs, it can be inferred that data from receive to vtablet_writer is slow, but can not judge the root cause is the scanner slow or is it slow to receive data. Finally, finding the network is too slow, but it costs lots of time. Therefore, recording the time on receiving HTTP request is helpful to users to know why the load is slow.
Observability effect
bvar
logs
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...