You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "zzzxl1993 (via GitHub)" <gi...@apache.org> on 2026/01/26 08:58:00 UTC
[PR] [fix](inverted-index) Fix boolean query AllScorer combination handling [doris]
zzzxl1993 opened a new pull request, #60237:
URL: https://github.com/apache/doris/pull/60237
### 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] [fix](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3832741552
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
Re: [PR] [fix](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3798860703
# BE UT Coverage Report
Increment line coverage `90.00% (45/50)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/00635aa407e66d347413817b3cb23c0790228ea2_00635aa407e66d347413817b3cb23c0790228ea2/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/00635aa407e66d347413817b3cb23c0790228ea2_00635aa407e66d347413817b3cb23c0790228ea2/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.47% (19142/36480) |
| Line Coverage | 35.85% (177881/496167) |
| Region Coverage | 32.31% (137435/425418) |
| Branch Coverage | 33.24% (59486/178964) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3798511266
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] [fix](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3822604890
<details>
<summary>TPC-H: <b>Total hot run time: 31893 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 a63b5f1996e051e165cbfa26711091a9480c6862, data reload: false
------ Round 1 ----------------------------------
q1 17649 5284 5131 5131
q2 2059 342 187 187
q3 10173 1293 733 733
q4 10232 902 316 316
q5 7548 2131 1952 1952
q6 192 180 147 147
q7 880 724 629 629
q8 9259 1371 1052 1052
q9 5354 4825 4787 4787
q10 6846 1962 1575 1575
q11 500 294 271 271
q12 327 372 225 225
q13 17809 4018 3203 3203
q14 235 235 211 211
q15 897 816 836 816
q16 688 683 627 627
q17 654 751 520 520
q18 6897 6522 6453 6453
q19 1215 981 625 625
q20 389 341 243 243
q21 2613 2038 1923 1923
q22 353 312 267 267
Total cold run time: 102769 ms
Total hot run time: 31893 ms
----- Round 2, with runtime_filter_mode=off -----
q1 5389 5351 5381 5351
q2 250 333 246 246
q3 2150 2710 2281 2281
q4 1385 1731 1338 1338
q5 4245 4168 4190 4168
q6 218 183 140 140
q7 2278 2092 1831 1831
q8 2668 2435 2435 2435
q9 7729 7580 7694 7580
q10 2976 3069 2819 2819
q11 554 523 468 468
q12 690 771 645 645
q13 3854 4531 3516 3516
q14 294 309 291 291
q15 892 826 825 825
q16 679 742 711 711
q17 1117 1340 1312 1312
q18 8139 8139 7964 7964
q19 863 817 825 817
q20 2077 2118 1905 1905
q21 4504 4188 4147 4147
q22 559 555 495 495
Total cold run time: 53510 ms
Total hot run time: 51285 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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3816158030
<details>
<summary>TPC-H: <b>Total hot run time: 32129 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 77b4f85ced06949966b9aca535db6d40af8e9421, data reload: false
------ Round 1 ----------------------------------
q1 17602 5310 5086 5086
q2 2221 312 196 196
q3 10065 1289 764 764
q4 10214 844 317 317
q5 7541 2181 1870 1870
q6 195 178 150 150
q7 889 743 597 597
q8 9267 1355 1156 1156
q9 5212 4822 4811 4811
q10 6801 1937 1566 1566
q11 525 301 275 275
q12 339 376 229 229
q13 17767 4108 3246 3246
q14 241 235 220 220
q15 896 812 833 812
q16 671 681 622 622
q17 632 802 481 481
q18 6678 6463 7445 6463
q19 1270 1068 649 649
q20 409 372 249 249
q21 3038 2322 2078 2078
q22 375 327 292 292
Total cold run time: 102848 ms
Total hot run time: 32129 ms
----- Round 2, with runtime_filter_mode=off -----
q1 5593 5565 5630 5565
q2 301 329 260 260
q3 2394 2822 2512 2512
q4 1507 1965 1423 1423
q5 4859 4557 4573 4557
q6 226 185 135 135
q7 2035 1925 1827 1827
q8 2741 2440 2401 2401
q9 7557 7291 7686 7291
q10 2815 2979 2606 2606
q11 703 577 442 442
q12 688 759 584 584
q13 3908 4018 3238 3238
q14 277 298 264 264
q15 840 795 797 795
q16 639 681 632 632
q17 1084 1264 1271 1264
q18 7311 7380 7331 7331
q19 846 828 796 796
q20 1974 2038 1901 1901
q21 4514 4254 4170 4170
q22 584 528 504 504
Total cold run time: 53396 ms
Total hot run time: 50498 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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3822800776
# BE UT Coverage Report
Increment line coverage `88.52% (54/61)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/a63b5f1996e051e165cbfa26711091a9480c6862_a63b5f1996e051e165cbfa26711091a9480c6862/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/a63b5f1996e051e165cbfa26711091a9480c6862_a63b5f1996e051e165cbfa26711091a9480c6862/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.50% (19278/36719) |
| Line Coverage | 35.97% (179083/497817) |
| Region Coverage | 32.40% (138966/428850) |
| Branch Coverage | 33.33% (60093/180285) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3832741616
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] [fix](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "airborne12 (via GitHub)" <gi...@apache.org>.
airborne12 merged PR #60237:
URL: https://github.com/apache/doris/pull/60237
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3798679731
<details>
<summary>ClickBench: <b>Total hot run time: 27.92 s</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 00635aa407e66d347413817b3cb23c0790228ea2, data reload: false
query1 0.05 0.04 0.04
query2 0.10 0.04 0.04
query3 0.26 0.08 0.10
query4 1.61 0.11 0.11
query5 0.27 0.24 0.25
query6 1.16 0.68 0.69
query7 0.02 0.02 0.03
query8 0.05 0.04 0.03
query9 0.57 0.49 0.49
query10 0.53 0.55 0.55
query11 0.13 0.09 0.09
query12 0.14 0.11 0.10
query13 0.63 0.61 0.61
query14 1.04 1.04 1.04
query15 0.89 0.87 0.87
query16 0.39 0.40 0.38
query17 1.04 1.07 1.12
query18 0.22 0.21 0.21
query19 1.99 2.00 1.90
query20 0.02 0.01 0.02
query21 15.45 0.27 0.14
query22 5.40 0.05 0.05
query23 16.20 0.27 0.10
query24 1.02 0.43 0.18
query25 0.08 0.09 0.06
query26 0.13 0.13 0.13
query27 0.09 0.06 0.06
query28 3.58 1.14 0.96
query29 12.56 3.95 3.16
query30 0.27 0.13 0.11
query31 2.82 0.62 0.40
query32 3.22 0.58 0.50
query33 3.16 3.30 3.22
query34 16.23 5.36 4.71
query35 4.79 4.71 4.81
query36 0.65 0.50 0.49
query37 0.10 0.06 0.06
query38 0.07 0.04 0.04
query39 0.05 0.03 0.03
query40 0.19 0.16 0.15
query41 0.09 0.04 0.03
query42 0.04 0.03 0.03
query43 0.05 0.04 0.04
Total cold run time: 97.35 s
Total hot run time: 27.92 s
```
</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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3822672321
<details>
<summary>ClickBench: <b>Total hot run time: 28.16 s</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit a63b5f1996e051e165cbfa26711091a9480c6862, data reload: false
query1 0.05 0.05 0.04
query2 0.11 0.04 0.04
query3 0.26 0.09 0.08
query4 1.61 0.11 0.11
query5 0.26 0.24 0.25
query6 1.18 0.67 0.67
query7 0.04 0.02 0.03
query8 0.06 0.03 0.04
query9 0.58 0.49 0.51
query10 0.56 0.55 0.54
query11 0.14 0.09 0.10
query12 0.14 0.10 0.11
query13 0.63 0.62 0.61
query14 1.05 1.06 1.08
query15 0.87 0.87 0.89
query16 0.39 0.39 0.40
query17 1.12 1.09 1.15
query18 0.24 0.21 0.21
query19 2.05 1.93 2.03
query20 0.02 0.01 0.02
query21 15.42 0.26 0.15
query22 5.16 0.05 0.05
query23 16.09 0.29 0.10
query24 1.92 0.56 0.19
query25 0.09 0.10 0.06
query26 0.15 0.14 0.13
query27 0.07 0.07 0.04
query28 4.20 1.15 0.97
query29 12.53 3.93 3.16
query30 0.28 0.14 0.12
query31 2.81 0.63 0.40
query32 3.27 0.59 0.50
query33 3.22 3.26 3.23
query34 16.31 5.39 4.77
query35 4.77 4.79 4.79
query36 0.66 0.50 0.49
query37 0.11 0.07 0.08
query38 0.08 0.05 0.04
query39 0.05 0.03 0.03
query40 0.20 0.17 0.15
query41 0.08 0.04 0.03
query42 0.05 0.04 0.03
query43 0.04 0.04 0.04
Total cold run time: 98.92 s
Total hot run time: 28.16 s
```
</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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "zzzxl1993 (via GitHub)" <gi...@apache.org>.
zzzxl1993 commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3822463455
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3823152581
# BE Regression && UT Coverage Report
Increment line coverage `96.72% (59/61)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60237_a63b5f1996e051e165cbfa26711091a9480c6862_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60237_a63b5f1996e051e165cbfa26711091a9480c6862_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.56% (25756/35994) |
| Line Coverage | 54.19% (269153/496640) |
| Region Coverage | 51.69% (223972/433260) |
| Branch Coverage | 53.12% (96153/181017) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3799145084
# BE Regression && UT Coverage Report
Increment line coverage `100.00% (50/50)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60237_00635aa407e66d347413817b3cb23c0790228ea2_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60237_00635aa407e66d347413817b3cb23c0790228ea2_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.42% (25537/35757) |
| Line Coverage | 54.03% (267785/495623) |
| Region Coverage | 51.62% (221907/429923) |
| Branch Coverage | 53.09% (95428/179732) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "zzzxl1993 (via GitHub)" <gi...@apache.org>.
zzzxl1993 commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3798511827
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3816900069
# BE Regression && UT Coverage Report
Increment line coverage `96.72% (59/61)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60237_77b4f85ced06949966b9aca535db6d40af8e9421_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60237_77b4f85ced06949966b9aca535db6d40af8e9421_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.46% (25597/35821) |
| Line Coverage | 54.07% (267402/494562) |
| Region Coverage | 51.80% (223123/430712) |
| Branch Coverage | 53.11% (95626/180067) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3816234772
<details>
<summary>ClickBench: <b>Total hot run time: 28.31 s</b></summary>
```
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 77b4f85ced06949966b9aca535db6d40af8e9421, data reload: false
query1 0.05 0.04 0.04
query2 0.10 0.04 0.04
query3 0.25 0.08 0.08
query4 1.61 0.10 0.11
query5 0.26 0.25 0.25
query6 1.18 0.67 0.67
query7 0.04 0.02 0.02
query8 0.05 0.04 0.04
query9 0.58 0.51 0.49
query10 0.54 0.56 0.53
query11 0.15 0.10 0.09
query12 0.14 0.11 0.11
query13 0.64 0.62 0.63
query14 1.07 1.05 1.07
query15 0.88 0.87 0.88
query16 0.38 0.40 0.42
query17 1.17 1.14 1.14
query18 0.23 0.22 0.21
query19 2.11 2.04 1.93
query20 0.02 0.01 0.01
query21 15.39 0.29 0.15
query22 4.94 0.05 0.05
query23 15.72 0.31 0.12
query24 1.74 0.25 0.29
query25 0.10 0.09 0.06
query26 0.15 0.13 0.13
query27 0.10 0.08 0.05
query28 3.29 1.16 0.97
query29 12.52 3.95 3.17
query30 0.28 0.14 0.11
query31 2.81 0.66 0.42
query32 3.24 0.60 0.49
query33 3.24 3.29 3.31
query34 16.58 5.34 4.74
query35 4.80 4.80 4.74
query36 0.65 0.50 0.49
query37 0.13 0.07 0.07
query38 0.06 0.04 0.04
query39 0.05 0.03 0.04
query40 0.19 0.16 0.15
query41 0.09 0.04 0.03
query42 0.04 0.03 0.03
query43 0.05 0.04 0.04
Total cold run time: 97.61 s
Total hot run time: 28.31 s
```
</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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3798619059
<details>
<summary>TPC-H: <b>Total hot run time: 32950 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 00635aa407e66d347413817b3cb23c0790228ea2, data reload: false
------ Round 1 ----------------------------------
q1 17653 5338 5099 5099
q2 2054 317 196 196
q3 10182 1316 764 764
q4 10217 913 323 323
q5 7529 2158 1907 1907
q6 202 180 151 151
q7 895 723 589 589
q8 9250 1406 1113 1113
q9 5234 4831 4946 4831
q10 6753 1939 1563 1563
q11 526 295 271 271
q12 341 379 232 232
q13 17787 4028 3191 3191
q14 231 233 212 212
q15 905 832 805 805
q16 675 713 622 622
q17 643 839 440 440
q18 6722 6514 7507 6514
q19 1260 1025 670 670
q20 409 367 254 254
q21 3053 2304 2134 2134
q22 1159 1069 1102 1069
Total cold run time: 103680 ms
Total hot run time: 32950 ms
----- Round 2, with runtime_filter_mode=off -----
q1 5548 5517 5932 5517
q2 270 340 250 250
q3 2397 2838 2589 2589
q4 1505 1880 1427 1427
q5 4544 4485 4651 4485
q6 223 181 145 145
q7 2061 1991 1769 1769
q8 2493 2480 2416 2416
q9 7495 7501 7556 7501
q10 2887 3028 2604 2604
q11 528 490 462 462
q12 715 752 638 638
q13 3812 4451 3356 3356
q14 260 284 272 272
q15 826 788 798 788
q16 637 676 639 639
q17 1084 1276 1303 1276
q18 7608 7376 7389 7376
q19 859 840 845 840
q20 1985 2025 1876 1876
q21 4534 4163 4134 4134
q22 1078 1023 984 984
Total cold run time: 53349 ms
Total hot run time: 51344 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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3824438413
# BE Regression && UT Coverage Report
Increment line coverage `96.72% (59/61)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60237_a63b5f1996e051e165cbfa26711091a9480c6862_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60237_a63b5f1996e051e165cbfa26711091a9480c6862_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.58% (25763/35994) |
| Line Coverage | 54.22% (269284/496640) |
| Region Coverage | 51.71% (224045/433260) |
| Branch Coverage | 53.13% (96177/181017) |
--
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "zzzxl1993 (via GitHub)" <gi...@apache.org>.
zzzxl1993 commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3815958358
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](inverted-index) Fix boolean query AllScorer combination handling [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60237:
URL: https://github.com/apache/doris/pull/60237#issuecomment-3816515759
# BE UT Coverage Report
Increment line coverage `88.52% (54/61)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/77b4f85ced06949966b9aca535db6d40af8e9421_77b4f85ced06949966b9aca535db6d40af8e9421/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/77b4f85ced06949966b9aca535db6d40af8e9421_77b4f85ced06949966b9aca535db6d40af8e9421/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.73% (19270/36543) |
| Line Coverage | 36.12% (179098/495795) |
| Region Coverage | 32.58% (138930/426394) |
| Branch Coverage | 33.50% (60087/179351) |
--
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