Uh oh!
There was an error while loading. Please reload this page.
[fix](pipelinex) fix fragment instance progress reports - #40325
Conversation
doris-robot
commented
Sep 3, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
kaijchen
commented
Sep 3, 2024
run buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
doris-robot
commented
Sep 3, 2024
TPC-H: Total hot run time: 38440 ms |
doris-robot
commented
Sep 3, 2024
TPC-DS: Total hot run time: 187205 ms |
doris-robot
commented
Sep 3, 2024
ClickBench: Total hot run time: 32.34 s |
clang-tidy review says "All clean, LGTM! 👍" |
kaijchen
commented
Sep 4, 2024
run buildall |
doris-robot
commented
Sep 4, 2024
TPC-H: Total hot run time: 38155 ms |
doris-robot
commented
Sep 4, 2024
TPC-DS: Total hot run time: 192550 ms |
doris-robot
commented
Sep 4, 2024
ClickBench: Total hot run time: 32.1 s |
kaijchen
commented
Sep 4, 2024
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Sep 4, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Sep 4, 2024
TPC-H: Total hot run time: 38797 ms |
doris-robot
commented
Sep 4, 2024
TPC-DS: Total hot run time: 188284 ms |
kaijchen
commented
Sep 5, 2024
run buildall |
doris-robot
commented
Sep 5, 2024
TPC-H: Total hot run time: 38143 ms |
doris-robot
commented
Sep 5, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Sep 5, 2024
TPC-DS: Total hot run time: 194854 ms |
doris-robot
commented
Sep 5, 2024
ClickBench: Total hot run time: 31.89 s |
PR approved by at least one committer and no changes requested. |
## Proposed changes Fix progress in `show load` not 100% when load finished. PipelineX has deprecated `fragmentInstanceId` in `TReportExecStatusParams`. The `finishedScanNums` should be reported with distinct `fragmentInstanceId` to prevent being overwritten. Before ``` | 38024 | lineitem_dup_1_replication_s3_2 | FINISHED | 30.00% (3/10) | BROKER | ... ``` After ``` | 39025 | lineitem_dup_1_replication_s3_3 | FINISHED | 100.00% (10/10) | BROKER | ... ```
Fix progress in `show load` not 100% when load finished. PipelineX has deprecated `fragmentInstanceId` in `TReportExecStatusParams`. The `finishedScanNums` should be reported with distinct `fragmentInstanceId` to prevent being overwritten. Before ``` | 38024 | lineitem_dup_1_replication_s3_2 | FINISHED | 30.00% (3/10) | BROKER | ... ``` After ``` | 39025 | lineitem_dup_1_replication_s3_3 | FINISHED | 100.00% (10/10) | BROKER | ... ```
) ## Proposed changes This PR is a followup of #40325. Because PipelineX has deprecated the old report. This PR fixed the `ScannedRows` and `LoadBytes` in the progress of the `SHOW LOAD` result. Currently the progress will only be updated when a fragment instance is done. Timely progress updates will be supported in a later PR.
) ## Proposed changes This PR is a followup of #40325. Because PipelineX has deprecated the old report. This PR fixed the `ScannedRows` and `LoadBytes` in the progress of the `SHOW LOAD` result. Currently the progress will only be updated when a fragment instance is done. Timely progress updates will be supported in a later PR.
…che#40694) This PR is a followup of apache#40325. Because PipelineX has deprecated the old report. This PR fixed the `ScannedRows` and `LoadBytes` in the progress of the `SHOW LOAD` result. Currently the progress will only be updated when a fragment instance is done. Timely progress updates will be supported in a later PR.
…che#40694) This PR is a followup of apache#40325. Because PipelineX has deprecated the old report. This PR fixed the `ScannedRows` and `LoadBytes` in the progress of the `SHOW LOAD` result. Currently the progress will only be updated when a fragment instance is done. Timely progress updates will be supported in a later PR.
## Proposed changes Fix progress in `show load` not 100% when load finished. PipelineX has deprecated `fragmentInstanceId` in `TReportExecStatusParams`. The `finishedScanNums` should be reported with distinct `fragmentInstanceId` to prevent being overwritten. Before ``` | 38024 | lineitem_dup_1_replication_s3_2 | FINISHED | 30.00% (3/10) | BROKER | ... ``` After ``` | 39025 | lineitem_dup_1_replication_s3_3 | FINISHED | 100.00% (10/10) | BROKER | ... ```
…che#40694) ## Proposed changes This PR is a followup of apache#40325. Because PipelineX has deprecated the old report. This PR fixed the `ScannedRows` and `LoadBytes` in the progress of the `SHOW LOAD` result. Currently the progress will only be updated when a fragment instance is done. Timely progress updates will be supported in a later PR.
Proposed changes
Fix progress in
show loadnot 100% when load finished.PipelineX has deprecated
fragmentInstanceIdinTReportExecStatusParams.The
finishedScanNumsshould be reported with distinctfragmentInstanceIdto prevent being overwritten.Before
After