-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Fix](autoinc) try fix concurrent load problem with auto inc column #36421
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
|
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! 👍" |
a771e13 to
38154b4
Compare
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
dataroaring
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 at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 39815 ms |
TPC-DS: Total hot run time: 173836 ms |
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 30.79 s |
zhannngchen
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.
pls add description for your patch
1f3c240 to
633dab0
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
3 similar comments
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
TPC-H: Total hot run time: 39823 ms |
TPC-DS: Total hot run time: 175165 ms |
ClickBench: Total hot run time: 30.6 s |
6cdebf6 to
c73fd0c
Compare
TPC-H: Total hot run time: 39732 ms |
TPC-DS: Total hot run time: 174800 ms |
ClickBench: Total hot run time: 31.37 s |
dataroaring
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
9891812 to
95f6750
Compare
|
run build all |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run build all |
|
run buildall |
TPC-H: Total hot run time: 40513 ms |
TPC-DS: Total hot run time: 174043 ms |
ClickBench: Total hot run time: 31.05 s |
zhannngchen
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 at least one committer and no changes requested. |
…36421) 1. increase the value of `AutoIncrementGenerator.BATCH_ID_INTERVAL` to reduce the number of writes to BDBJE. (the default value of `config::auto_inc_prefetch_size_ratio` is 10 and the default value of `AutoIncIDBuffer::_batch_size` is 4064, so mostly the request length is 40960) 2. only allow master fe to offer `getAutoIncrementRange` service 3. write editlog before update `batchEndId` in memory in `getAutoIncrementRange ` 4. refactor `AutoIncIDBuffer`
AutoIncrementGenerator.BATCH_ID_INTERVALto reduce the number of writes to BDBJE. (the default value ofconfig::auto_inc_prefetch_size_ratiois 10 and the default value ofAutoIncIDBuffer::_batch_sizeis 4064, so mostly the request length is 40960)getAutoIncrementRangeservicebatchEndIdin memory ingetAutoIncrementRangeAutoIncIDBufferbranch-2.1 pick: #37027