[enhance](load) only set brokerLoadBatchSize when enableMemtableOnSinkNode is true#60301
[enhance](load) only set brokerLoadBatchSize when enableMemtableOnSinkNode is true#60301JNSimba merged 1 commit intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
13593a8 to
d627f30
Compare
|
run buildall |
d627f30 to
5864b57
Compare
|
run buildall |
TPC-H: Total hot run time: 32253 ms |
ClickBench: Total hot run time: 28.31 s |
FE UT Coverage ReportIncrement line coverage |
FE Regression Coverage ReportIncrement line coverage |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
5864b57 to
baac91b
Compare
|
run buildall |
|
run buildall |
TPC-H: Total hot run time: 32180 ms |
ClickBench: Total hot run time: 28.22 s |
01bf8cb to
64eca84
Compare
|
run buildall |
TPC-H: Total hot run time: 31974 ms |
ClickBench: Total hot run time: 28.41 s |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
|
run buildall |
3d96876 to
abe0ec6
Compare
|
run buildall |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
1 similar comment
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 31888 ms |
ClickBench: Total hot run time: 28.1 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
FE Regression Coverage ReportIncrement line coverage |
1aa5fca to
e7b1943
Compare
|
run buildall |
TPC-H: Total hot run time: 31955 ms |
ClickBench: Total hot run time: 28.23 s |
FE Regression Coverage ReportIncrement line coverage |
1 similar comment
FE Regression Coverage ReportIncrement line coverage |
|
PR approved by at least one committer and no changes requested. |
…kNode is true (#60301) ### What problem does this PR solve? #36477 introduce brokerLoadBatchSize to improve performance for greater batch size. But for load path in `VNodeChannel`(`enableMemtableOnSinkNode` is false), the batching operation has been performed when sending the block, at the same time, memory issue was introduced: large memory occupation happened when serialize/deserialize block in wide-table scenarios. **Serialize block:** <img width="312" height="588" alt="image" src="https://github.com/user-attachments/assets/e63390cb-83ac-4cd8-b171-6fb6fc5a2785" /> **Deserialize block:** <img width="312" height="488" alt="image" src="https://github.com/user-attachments/assets/425d136d-21b0-4c8d-a29f-1686d3de4fcb" /> To solve this problem, only set brokerLoadBatchSize when enableMemtableOnSinkNode is true
What problem does this PR solve?
#36477 introduce brokerLoadBatchSize to improve performance for greater batch size. But for load path in
VNodeChannel(enableMemtableOnSinkNodeis false), the batching operation has been performed when sending the block, at the same time, memory issue was introduced: large memory occupation happened when serialize/deserialize block in wide-table scenarios.Serialize block:

Deserialize block:

To solve this problem, only set brokerLoadBatchSize when enableMemtableOnSinkNode is true
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)