You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "HappenLee (via GitHub)" <gi...@apache.org> on 2026/03/03 04:31:17 UTC
[PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
HappenLee opened a new pull request, #60970:
URL: https://github.com/apache/doris/pull/60970
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg: https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "Gabriel39 (via GitHub)" <gi...@apache.org>.
Gabriel39 merged PR #60970:
URL: https://github.com/apache/doris/pull/60970
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4116172177
# BE UT Coverage Report
Increment line coverage ` ` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/9288f23c682b6312429e06a4ce5feb2c482d474f_9288f23c682b6312429e06a4ce5feb2c482d474f/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/9288f23c682b6312429e06a4ce5feb2c482d474f_9288f23c682b6312429e06a4ce5feb2c482d474f/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.79% (19857/37615) |
| Line Coverage | 36.29% (185522/511171) |
| Region Coverage | 32.54% (143604/441342) |
| Branch Coverage | 33.76% (62898/186312) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3988980908
<details>
<summary>TPC-DS: <b>Total hot run time: 184578 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 1559eefee5715a4aa4f19ba45c1853dfefc9841e, data reload: false
query5 4337 651 534 534
query6 335 215 202 202
query7 4205 464 271 271
query8 339 244 239 239
query9 8733 2789 2791 2789
query10 534 384 350 350
query11 17058 17023 16673 16673
query12 186 133 129 129
query13 1264 453 364 364
query14 6331 3257 3013 3013
query14_1 2896 2801 2839 2801
query15 199 192 179 179
query16 988 458 454 454
query17 1111 725 624 624
query18 2510 451 356 356
query19 211 210 188 188
query20 143 132 131 131
query21 222 151 123 123
query22 5042 5504 5462 5462
query23 17619 17180 16889 16889
query23_1 16990 17384 16974 16974
query24 7355 1613 1218 1218
query24_1 1225 1243 1240 1240
query25 533 442 401 401
query26 1240 267 149 149
query27 2758 464 286 286
query28 4518 1889 1887 1887
query29 805 553 474 474
query30 313 253 211 211
query31 855 727 648 648
query32 83 79 71 71
query33 514 332 286 286
query34 947 912 561 561
query35 641 669 589 589
query36 1069 1124 960 960
query37 138 92 84 84
query38 2937 2993 2820 2820
query39 882 870 849 849
query39_1 831 880 819 819
query40 231 151 135 135
query41 65 64 60 60
query42 110 104 101 101
query43 368 378 354 354
query44
query45 197 191 180 180
query46 888 986 626 626
query47 2094 2114 2027 2027
query48 305 332 237 237
query49 639 464 395 395
query50 673 294 223 223
query51 4076 4123 4020 4020
query52 107 107 95 95
query53 294 342 288 288
query54 296 266 263 263
query55 92 88 81 81
query56 324 322 302 302
query57 1362 1330 1272 1272
query58 296 276 275 275
query59 2704 2795 2544 2544
query60 346 333 315 315
query61 155 148 183 148
query62 629 581 526 526
query63 314 283 278 278
query64 4947 1280 1002 1002
query65
query66 1446 467 352 352
query67 16337 16497 16279 16279
query68
query69 404 298 286 286
query70 988 914 951 914
query71 336 310 324 310
query72 2792 2702 2480 2480
query73 550 545 325 325
query74 10007 9871 9770 9770
query75 2871 2777 2494 2494
query76 2269 1062 692 692
query77 379 397 325 325
query78 11132 11329 10680 10680
query79 1151 796 599 599
query80 1348 647 576 576
query81 562 286 257 257
query82 1010 152 118 118
query83 336 272 260 260
query84 255 121 104 104
query85 970 574 513 513
query86 418 319 309 309
query87 3139 3156 3021 3021
query88 3610 2732 2679 2679
query89 422 370 350 350
query90 1988 181 173 173
query91 158 154 135 135
query92 74 77 68 68
query93 927 836 510 510
query94 631 328 294 294
query95 594 412 330 330
query96 663 513 234 234
query97 2493 2505 2421 2421
query98 232 219 226 219
query99 1007 1034 900 900
Total cold run time: 253267 ms
Total hot run time: 184578 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4123789150
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.36% (21128/36832) |
| Line Coverage | 40.39% (205839/509619) |
| Region Coverage | 37.27% (166032/445482) |
| Branch Coverage | 38.08% (71161/186880) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990922889
<details>
<summary>TPC-DS: <b>Total hot run time: 183614 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 8d0836b792a8c06009d09c8ab2049418d32e3e78, data reload: false
query5 4325 628 514 514
query6 316 217 218 217
query7 4231 473 268 268
query8 335 247 238 238
query9 8734 2757 2812 2757
query10 511 402 336 336
query11 16997 17325 17209 17209
query12 181 135 153 135
query13 1361 477 347 347
query14 6546 3388 3039 3039
query14_1 2979 2965 2890 2890
query15 203 192 182 182
query16 999 489 490 489
query17 1203 749 628 628
query18 2996 486 361 361
query19 201 212 187 187
query20 143 152 136 136
query21 215 151 123 123
query22 5199 5142 4591 4591
query23 17156 16757 16510 16510
query23_1 16741 16628 16632 16628
query24 7217 1608 1217 1217
query24_1 1239 1250 1256 1250
query25 534 456 400 400
query26 1224 252 149 149
query27 2792 476 291 291
query28 4520 1879 1877 1877
query29 820 566 476 476
query30 316 250 212 212
query31 863 742 650 650
query32 77 69 71 69
query33 505 334 285 285
query34 920 908 559 559
query35 623 691 584 584
query36 1063 1133 960 960
query37 135 94 86 86
query38 2986 2949 2864 2864
query39 890 880 855 855
query39_1 828 838 818 818
query40 235 160 138 138
query41 69 68 64 64
query42 108 105 105 105
query43 372 389 365 365
query44
query45 199 192 188 188
query46 898 997 632 632
query47 2125 2165 2091 2091
query48 330 331 236 236
query49 648 485 394 394
query50 697 282 218 218
query51 4106 4127 4048 4048
query52 109 109 100 100
query53 295 354 293 293
query54 330 289 273 273
query55 95 87 86 86
query56 323 340 336 336
query57 1359 1348 1275 1275
query58 298 285 290 285
query59 2592 2751 2557 2557
query60 345 352 344 344
query61 169 171 171 171
query62 629 605 546 546
query63 322 282 287 282
query64 5078 1343 1059 1059
query65
query66 1487 465 365 365
query67 16788 16489 16299 16299
query68
query69 393 328 309 309
query70 1053 1014 970 970
query71 347 320 302 302
query72 2947 2644 2390 2390
query73 535 564 319 319
query74 9999 9915 9715 9715
query75 2861 2743 2447 2447
query76 2283 1025 705 705
query77 371 384 317 317
query78 11149 11439 10642 10642
query79 2439 788 625 625
query80 1808 622 537 537
query81 570 284 248 248
query82 971 160 118 118
query83 338 266 242 242
query84 254 115 102 102
query85 887 470 430 430
query86 442 295 300 295
query87 3106 3110 2974 2974
query88 3572 2696 2638 2638
query89 429 379 349 349
query90 2006 178 175 175
query91 164 167 144 144
query92 78 69 69 69
query93 1176 825 506 506
query94 639 325 301 301
query95 575 343 383 343
query96 644 505 230 230
query97 2438 2476 2397 2397
query98 231 217 215 215
query99 1013 986 893 893
Total cold run time: 256081 ms
Total hot run time: 183614 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4125176577
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.34% (21118/36832) |
| Line Coverage | 40.38% (205760/509619) |
| Region Coverage | 37.25% (165950/445482) |
| Branch Coverage | 38.08% (71164/186880) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990826880
<details>
<summary>TPC-H: <b>Total hot run time: 28769 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 8d0836b792a8c06009d09c8ab2049418d32e3e78, data reload: false
------ Round 1 ----------------------------------
============================================
q1 17633 4392 4267 4267
q2 q3 10649 786 516 516
q4 4670 363 248 248
q5 7549 1188 1026 1026
q6 180 175 144 144
q7 773 830 656 656
q8 9397 1465 1292 1292
q9 4956 4770 4732 4732
q10 6870 1871 1650 1650
q11 465 271 266 266
q12 743 631 475 475
q13 17769 4232 3408 3408
q14 228 230 211 211
q15 945 790 811 790
q16 755 727 675 675
q17 730 864 425 425
q18 5920 5473 5272 5272
q19 1424 991 610 610
q20 508 505 389 389
q21 5043 1909 1428 1428
q22 380 342 289 289
Total cold run time: 97587 ms
Total hot run time: 28769 ms
----- Round 2, with runtime_filter_mode=off -----
============================================
q1 4629 4598 4671 4598
q2 q3 1846 2196 1777 1777
q4 875 1201 762 762
q5 4058 4509 4369 4369
q6 188 177 142 142
q7 1765 1642 1544 1544
q8 2506 2700 2558 2558
q9 7493 7312 7333 7312
q10 2724 2849 2488 2488
q11 514 444 416 416
q12 498 584 467 467
q13 4230 4408 3570 3570
q14 283 292 274 274
q15 858 851 828 828
q16 752 829 741 741
q17 1302 1583 1320 1320
q18 7198 6862 6639 6639
q19 887 891 865 865
q20 2113 2197 2017 2017
q21 4034 3499 3376 3376
q22 451 421 372 372
Total cold run time: 49204 ms
Total hot run time: 46435 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "HappenLee (via GitHub)" <gi...@apache.org>.
HappenLee commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4115859509
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990399105
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 56.69% (20733/36574) |
| Line Coverage | 39.77% (200826/504991) |
| Region Coverage | 36.17% (159812/441873) |
| Branch Coverage | 36.94% (68332/184998) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3992582638
# BE Regression && UT Coverage Report
Increment line coverage `16.67% (1/6)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_8d0836b792a8c06009d09c8ab2049418d32e3e78_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_8d0836b792a8c06009d09c8ab2049418d32e3e78_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.53% (26163/36576) |
| Line Coverage | 54.30% (274223/505011) |
| Region Coverage | 51.82% (228979/441872) |
| Branch Coverage | 53.03% (98108/185006) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3989437371
# BE UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/1559eefee5715a4aa4f19ba45c1853dfefc9841e_1559eefee5715a4aa4f19ba45c1853dfefc9841e/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/1559eefee5715a4aa4f19ba45c1853dfefc9841e_1559eefee5715a4aa4f19ba45c1853dfefc9841e/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.57% (19636/37352) |
| Line Coverage | 36.18% (183288/506545) |
| Region Coverage | 32.47% (142133/437740) |
| Branch Coverage | 33.44% (61675/184438) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990312989
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 51.11% (18694/36573) |
| Line Coverage | 35.03% (176882/504962) |
| Region Coverage | 30.95% (136768/441852) |
| Branch Coverage | 32.23% (59612/184980) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4111286141
<details>
<summary>TPC-H: <b>Total hot run time: 26731 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 4d7a7f3e32781c1f2b763df0dca6a87d7cb4b87a, data reload: false
------ Round 1 ----------------------------------
orders Doris NULL NULL 0 0 0 NULL 0 NULL NULL 2023-12-26 18:27:23 2023-12-26 18:42:55 NULL utf-8 NULL NULL
============================================
q1 17651 4486 4312 4312
q2 q3 10641 787 534 534
q4 4675 353 244 244
q5 7575 1192 1035 1035
q6 175 173 147 147
q7 770 861 659 659
q8 9493 1477 1319 1319
q9 5175 4692 4688 4688
q10 6319 1914 1661 1661
q11 462 253 246 246
q12 748 605 473 473
q13 18037 2915 2205 2205
q14 228 237 213 213
q15 q16 756 744 687 687
q17 728 837 439 439
q18 5834 5363 5166 5166
q19 1323 976 615 615
q20 521 488 363 363
q21 4607 1839 1436 1436
q22 505 366 289 289
Total cold run time: 96223 ms
Total hot run time: 26731 ms
----- Round 2, with runtime_filter_mode=off -----
orders Doris NULL NULL 150000000 42 6422171781 NULL 22778155 NULL NULL 2023-12-26 18:27:23 2023-12-26 18:42:55 NULL utf-8 NULL NULL
============================================
q1 4827 4573 4660 4573
q2 q3 3876 4346 3806 3806
q4 942 1236 801 801
q5 4079 4379 4424 4379
q6 204 187 149 149
q7 1801 1627 1504 1504
q8 2471 2725 2528 2528
q9 7599 7389 7236 7236
q10 3726 3942 3614 3614
q11 543 465 425 425
q12 508 614 451 451
q13 2689 3337 2301 2301
q14 286 291 307 291
q15 q16 717 757 712 712
q17 1282 1353 1336 1336
q18 7210 6809 6566 6566
q19 886 936 929 929
q20 2050 2204 1999 1999
q21 3965 3478 3305 3305
q22 460 426 380 380
Total cold run time: 50121 ms
Total hot run time: 47285 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4111365808
<details>
<summary>TPC-DS: <b>Total hot run time: 168873 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 4d7a7f3e32781c1f2b763df0dca6a87d7cb4b87a, data reload: false
query5 4343 634 498 498
query6 349 228 216 216
query7 4235 487 264 264
query8 350 250 229 229
query9 8721 2724 2703 2703
query10 534 382 352 352
query11 6987 5099 4909 4909
query12 184 131 124 124
query13 1285 470 353 353
query14 5676 3711 3493 3493
query14_1 2768 2766 2735 2735
query15 200 191 171 171
query16 982 482 464 464
query17 1020 719 601 601
query18 2436 436 343 343
query19 207 208 178 178
query20 134 127 133 127
query21 213 130 107 107
query22 13316 14318 15018 14318
query23 16396 15832 15676 15676
query23_1 15940 16103 15724 15724
query24 7313 1642 1211 1211
query24_1 1227 1254 1246 1246
query25 580 491 439 439
query26 1235 272 154 154
query27 2758 486 291 291
query28 4513 1846 1819 1819
query29 851 563 481 481
query30 292 216 192 192
query31 998 940 868 868
query32 78 69 71 69
query33 514 347 301 301
query34 925 870 540 540
query35 639 678 595 595
query36 1107 1144 987 987
query37 134 92 84 84
query38 2926 2908 2924 2908
query39 847 843 821 821
query39_1 806 790 774 774
query40 228 145 133 133
query41 63 58 63 58
query42 262 258 254 254
query43 232 251 227 227
query44
query45 200 189 179 179
query46 878 969 622 622
query47 2153 2170 2070 2070
query48 322 320 232 232
query49 644 496 380 380
query50 694 269 209 209
query51 4078 3994 4009 3994
query52 261 264 254 254
query53 291 341 301 301
query54 318 310 272 272
query55 89 89 85 85
query56 323 306 320 306
query57 1942 1797 1592 1592
query58 288 298 267 267
query59 2830 2961 2760 2760
query60 341 335 317 317
query61 156 154 152 152
query62 624 581 545 545
query63 312 285 275 275
query64 5105 1281 993 993
query65
query66 1456 454 366 366
query67 24185 24313 24185 24185
query68
query69 398 306 304 304
query70 955 980 921 921
query71 334 301 291 291
query72 2773 2745 2659 2659
query73 546 551 315 315
query74 9588 9581 9357 9357
query75 2853 2772 2474 2474
query76 2290 1027 686 686
query77 367 383 317 317
query78 10890 11103 10429 10429
query79 1126 782 574 574
query80 1359 646 571 571
query81 541 262 225 225
query82 1017 188 124 124
query83 355 258 253 253
query84 271 120 103 103
query85 915 497 455 455
query86 419 324 283 283
query87 3140 3075 3035 3035
query88 3529 2641 2647 2641
query89 421 363 340 340
query90 2035 179 176 176
query91 171 163 138 138
query92 78 75 69 69
query93 958 827 495 495
query94 625 318 296 296
query95 581 348 381 348
query96 639 510 231 231
query97 2451 2494 2418 2418
query98 234 217 218 217
query99 1019 985 889 889
Total cold run time: 249582 ms
Total hot run time: 168873 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4123637327
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.36% (21126/36832) |
| Line Coverage | 40.39% (205821/509619) |
| Region Coverage | 37.24% (165891/445482) |
| Branch Coverage | 38.07% (71151/186880) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4132533987
PR approved by anyone and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4115986483
<details>
<summary>TPC-H: <b>Total hot run time: 26379 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 9288f23c682b6312429e06a4ce5feb2c482d474f, data reload: false
------ Round 1 ----------------------------------
orders Doris NULL NULL 0 0 0 NULL 0 NULL NULL 2023-12-26 18:27:23 2023-12-26 18:42:55 NULL utf-8 NULL NULL
============================================
q1 17637 4415 4284 4284
q2 q3 10647 754 514 514
q4 4683 349 247 247
q5 7566 1200 1031 1031
q6 176 171 144 144
q7 800 826 667 667
q8 9547 1473 1302 1302
q9 5421 4767 4669 4669
q10 6323 1916 1649 1649
q11 470 248 245 245
q12 736 581 463 463
q13 18033 2673 1925 1925
q14 228 232 215 215
q15 q16 734 747 657 657
q17 727 861 417 417
q18 5857 5439 5275 5275
q19 1306 974 611 611
q20 547 485 377 377
q21 4492 1818 1384 1384
q22 466 404 303 303
Total cold run time: 96396 ms
Total hot run time: 26379 ms
----- Round 2, with runtime_filter_mode=off -----
orders Doris NULL NULL 150000000 42 6422171781 NULL 22778155 NULL NULL 2023-12-26 18:27:23 2023-12-26 18:42:55 NULL utf-8 NULL NULL
============================================
q1 4779 4674 4679 4674
q2 q3 3852 4295 3838 3838
q4 904 1234 783 783
q5 4048 4318 4375 4318
q6 191 174 137 137
q7 1770 1668 1523 1523
q8 2548 2737 2602 2602
q9 7511 7426 7365 7365
q10 3732 4062 3671 3671
q11 527 445 426 426
q12 489 577 437 437
q13 2411 2868 2191 2191
q14 321 305 285 285
q15 q16 758 761 700 700
q17 1286 1541 1349 1349
q18 7282 6835 6621 6621
q19 886 911 904 904
q20 2154 2222 1977 1977
q21 3973 3514 3285 3285
q22 498 426 373 373
Total cold run time: 49920 ms
Total hot run time: 47459 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "HappenLee (via GitHub)" <gi...@apache.org>.
HappenLee commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4111052278
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "HappenLee (via GitHub)" <gi...@apache.org>.
HappenLee commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3988601949
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "HappenLee (via GitHub)" <gi...@apache.org>.
HappenLee commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990608494
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3991812392
# BE Regression && UT Coverage Report
Increment line coverage `16.67% (1/6)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_8d0836b792a8c06009d09c8ab2049418d32e3e78_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_8d0836b792a8c06009d09c8ab2049418d32e3e78_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.53% (26161/36576) |
| Line Coverage | 54.30% (274200/505011) |
| Region Coverage | 51.80% (228904/441872) |
| Branch Coverage | 53.03% (98101/185006) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990491506
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.02% (20855/36574) |
| Line Coverage | 40.10% (202516/504991) |
| Region Coverage | 36.54% (161454/441873) |
| Branch Coverage | 37.34% (69073/184998) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3991284508
# BE UT Coverage Report
Increment line coverage `16.67% (1/6)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/8d0836b792a8c06009d09c8ab2049418d32e3e78_8d0836b792a8c06009d09c8ab2049418d32e3e78/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/8d0836b792a8c06009d09c8ab2049418d32e3e78_8d0836b792a8c06009d09c8ab2049418d32e3e78/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.57% (19637/37354) |
| Line Coverage | 36.18% (183296/506564) |
| Region Coverage | 32.49% (142215/437735) |
| Branch Coverage | 33.45% (61699/184442) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3988939957
<details>
<summary>TPC-H: <b>Total hot run time: 28908 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 1559eefee5715a4aa4f19ba45c1853dfefc9841e, data reload: false
------ Round 1 ----------------------------------
============================================
q1 17650 4539 4328 4328
q2 q3 10645 827 542 542
q4 4694 366 253 253
q5 7850 1196 1039 1039
q6 197 176 151 151
q7 826 851 684 684
q8 10621 1491 1310 1310
q9 5788 4827 4765 4765
q10 6849 1915 1622 1622
q11 496 243 252 243
q12 752 576 468 468
q13 17842 4228 3430 3430
q14 231 235 211 211
q15 973 801 788 788
q16 772 717 667 667
q17 724 873 462 462
q18 6341 5347 5226 5226
q19 1137 984 633 633
q20 523 503 409 409
q21 4494 1876 1430 1430
q22 347 293 247 247
Total cold run time: 99752 ms
Total hot run time: 28908 ms
----- Round 2, with runtime_filter_mode=off -----
============================================
q1 4491 4363 4373 4363
q2 q3 1761 2166 1719 1719
q4 838 1163 780 780
q5 4043 4359 4321 4321
q6 185 177 144 144
q7 1730 1584 1459 1459
q8 2410 2696 2524 2524
q9 7857 7479 7274 7274
q10 2671 2879 2477 2477
q11 529 459 431 431
q12 495 616 472 472
q13 3993 4415 3715 3715
q14 280 325 292 292
q15 840 823 857 823
q16 713 753 694 694
q17 1148 1541 1332 1332
q18 7174 6762 6549 6549
q19 938 973 971 971
q20 2091 2194 2217 2194
q21 4052 3615 3362 3362
q22 434 426 398 398
Total cold run time: 48673 ms
Total hot run time: 46294 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3989473360
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 51.12% (18696/36574) |
| Line Coverage | 35.03% (176913/504992) |
| Region Coverage | 30.96% (136792/441877) |
| Branch Coverage | 32.23% (59627/185002) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3990590072
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (2/2)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_1559eefee5715a4aa4f19ba45c1853dfefc9841e_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.13% (20894/36574) |
| Line Coverage | 40.19% (202957/504991) |
| Region Coverage | 36.66% (161981/441873) |
| Branch Coverage | 37.46% (69292/184998) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4116034422
<details>
<summary>TPC-DS: <b>Total hot run time: 168954 ms</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 9288f23c682b6312429e06a4ce5feb2c482d474f, data reload: false
query5 4325 642 510 510
query6 318 234 215 215
query7 4228 459 262 262
query8 347 238 222 222
query9 8705 2691 2660 2660
query10 520 408 350 350
query11 6959 5094 4815 4815
query12 172 128 121 121
query13 1254 457 332 332
query14 5687 3703 3418 3418
query14_1 2828 2769 2765 2765
query15 205 191 171 171
query16 969 474 450 450
query17 867 703 595 595
query18 2448 438 345 345
query19 244 205 186 186
query20 134 126 133 126
query21 207 134 104 104
query22 13189 14067 14806 14067
query23 16792 16598 15911 15911
query23_1 15881 15798 15815 15798
query24 7191 1639 1243 1243
query24_1 1245 1246 1280 1246
query25 616 505 439 439
query26 1245 295 162 162
query27 2733 484 309 309
query28 4518 1908 1942 1908
query29 898 568 470 470
query30 302 224 188 188
query31 1016 945 866 866
query32 85 71 73 71
query33 502 337 287 287
query34 887 869 538 538
query35 631 674 590 590
query36 1107 1116 1011 1011
query37 138 94 80 80
query38 2975 2863 2831 2831
query39 853 825 808 808
query39_1 797 800 815 800
query40 227 158 136 136
query41 63 60 58 58
query42 263 262 255 255
query43 239 248 220 220
query44
query45 197 189 181 181
query46 874 969 602 602
query47 2161 2143 2064 2064
query48 314 312 231 231
query49 629 458 380 380
query50 699 272 208 208
query51 4103 4004 4031 4004
query52 269 267 254 254
query53 295 340 284 284
query54 306 278 266 266
query55 94 88 85 85
query56 316 328 318 318
query57 1906 1831 1677 1677
query58 284 272 271 271
query59 2802 2946 2739 2739
query60 352 344 334 334
query61 155 149 156 149
query62 604 594 538 538
query63 305 284 285 284
query64 5062 1276 1002 1002
query65
query66 1476 464 363 363
query67 24279 24403 24169 24169
query68
query69 403 314 283 283
query70 881 972 933 933
query71 332 311 302 302
query72 2852 2893 2651 2651
query73 546 549 314 314
query74 9658 9622 9452 9452
query75 2853 2774 2488 2488
query76 2291 1043 673 673
query77 370 396 313 313
query78 10904 11047 10492 10492
query79 2697 735 571 571
query80 1795 658 588 588
query81 553 258 229 229
query82 998 153 120 120
query83 341 277 255 255
query84 305 123 99 99
query85 989 507 456 456
query86 410 312 333 312
query87 3159 3136 2978 2978
query88 3518 2652 2639 2639
query89 419 363 343 343
query90 2016 175 176 175
query91 164 168 139 139
query92 80 75 72 72
query93 1128 827 502 502
query94 640 311 289 289
query95 582 388 315 315
query96 640 517 230 230
query97 2462 2466 2424 2424
query98 235 223 238 223
query99 1029 993 933 933
Total cold run time: 251706 ms
Total hot run time: 168954 ms
```
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4116627180
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.36% (21126/36832) |
| Line Coverage | 40.39% (205821/509619) |
| Region Coverage | 37.24% (165891/445482) |
| Branch Coverage | 38.07% (71151/186880) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [fix](materialization) Make merge_multi_response detect insufficient backend rows and return error instead of DCHECK [doris]
Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-3988593258
Thank you for your contribution to Apache Doris.
Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR).
Please clearly describe your PR:
1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
3. What features were added. Why was this function added?
4. Which code was refactored and why was this part of the code refactored?
5. Which functions were optimized and what is the difference before and after the optimization?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4125012388
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60970_9288f23c682b6312429e06a4ce5feb2c482d474f_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 57.36% (21128/36832) |
| Line Coverage | 40.39% (205839/509619) |
| Region Coverage | 37.27% (166032/445482) |
| Branch Coverage | 38.08% (71161/186880) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Exec](materialize) change the materialization_opertor logic of gc id_map [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60970:
URL: https://github.com/apache/doris/pull/60970#issuecomment-4132533781
PR approved by at least one committer and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org