-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Fix](Streamingjob) fix postgres incr consumer too slow #59919
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
Conversation
|
run buildall |
|
run buildall |
TPC-H: Total hot run time: 31466 ms |
TPC-DS: Total hot run time: 174775 ms |
ClickBench: Total hot run time: 26.63 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
|
run buildall |
2be9b48 to
6aef060
Compare
|
rum buildall |
TPC-H: Total hot run time: 31949 ms |
TPC-DS: Total hot run time: 173647 ms |
|
run buildall |
TPC-H: Total hot run time: 31706 ms |
TPC-DS: Total hot run time: 173659 ms |
ClickBench: Total hot run time: 26.77 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
sollhui
left a comment
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 anyone and no changes requested. |
…oris into fix_incr_consumer_slow
|
run buildall |
TPC-H: Total hot run time: 31567 ms |
TPC-H: Total hot run time: 31324 ms |
TPC-DS: Total hot run time: 173858 ms |
ClickBench: Total hot run time: 26.93 s |
|
run buildall |
TPC-H: Total hot run time: 31764 ms |
TPC-DS: Total hot run time: 175164 ms |
ClickBench: Total hot run time: 27.11 s |
|
run p0 |
|
PR approved by at least one committer and no changes requested. |
|
run nonConcurrent |
FE Regression Coverage ReportIncrement line coverage |
### What problem does this PR solve? Issue Number: close #xxx Related PR: #59798 If you close the reader before committing the LSN, the commit will not be completed.
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #59798
If you close the reader before committing the LSN, the commit will not be completed.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)