You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "csun5285 (via GitHub)" <gi...@apache.org> on 2026/03/12 06:46:32 UTC
[PR] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
csun5285 opened a new pull request, #61250:
URL: https://github.com/apache/doris/pull/61250
### 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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4052476848
<details>
<summary>TPC-H: <b>Total hot run time: 27692 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 6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c, data reload: false
------ Round 1 ----------------------------------
============================================
q1 17660 4416 4283 4283
q2 q3 10636 811 529 529
q4 4677 362 254 254
q5 7559 1196 1026 1026
q6 178 180 151 151
q7 835 835 678 678
q8 9903 1472 1317 1317
q9 5539 4724 4632 4632
q10 6325 1928 1662 1662
q11 480 248 240 240
q12 744 562 474 474
q13 18081 2916 2212 2212
q14 240 229 210 210
q15 941 830 817 817
q16 757 741 708 708
q17 711 871 409 409
q18 5973 5411 5282 5282
q19 1110 1020 627 627
q20 498 489 386 386
q21 4604 2001 1546 1546
q22 388 334 249 249
Total cold run time: 97839 ms
Total hot run time: 27692 ms
----- Round 2, with runtime_filter_mode=off -----
============================================
q1 4692 4658 4593 4593
q2 q3 3901 4360 3852 3852
q4 891 1187 787 787
q5 4059 4399 4441 4399
q6 188 188 142 142
q7 1794 1633 1539 1539
q8 2516 2757 2626 2626
q9 7507 7316 7540 7316
q10 3763 4112 3651 3651
q11 543 448 408 408
q12 480 605 456 456
q13 2765 3265 2299 2299
q14 284 298 277 277
q15 841 894 854 854
q16 728 825 728 728
q17 1202 1476 1450 1450
q18 7254 6832 6508 6508
q19 882 850 885 850
q20 2092 2183 2005 2005
q21 3953 3507 3308 3308
q22 453 425 398 398
Total cold run time: 50788 ms
Total hot run time: 48446 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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4052509392
<details>
<summary>TPC-DS: <b>Total hot run time: 153077 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 6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c, data reload: false
query5 4317 638 511 511
query6 326 242 215 215
query7 4233 481 276 276
query8 347 253 247 247
query9 8708 2776 2750 2750
query10 555 386 342 342
query11 7387 5797 5639 5639
query12 181 130 126 126
query13 1272 480 351 351
query14 5691 3782 3576 3576
query14_1 2861 2857 2825 2825
query15 200 196 177 177
query16 988 490 466 466
query17 923 732 633 633
query18 2441 454 368 368
query19 220 210 191 191
query20 135 137 133 133
query21 223 142 124 124
query22 4824 4826 4536 4536
query23 16862 16162 15755 15755
query23_1 15841 15845 15757 15757
query24 7311 1716 1277 1277
query24_1 1277 1282 1306 1282
query25 596 621 556 556
query26 1820 275 166 166
query27 2813 549 293 293
query28 4455 1875 1840 1840
query29 885 567 467 467
query30 311 245 212 212
query31 1362 1284 1217 1217
query32 87 77 76 76
query33 493 322 272 272
query34 902 936 557 557
query35 644 663 595 595
query36 1087 1124 903 903
query37 129 94 82 82
query38 2955 2947 2885 2885
query39 900 860 853 853
query39_1 807 827 821 821
query40 243 153 131 131
query41 63 59 58 58
query42 300 305 295 295
query43 256 249 225 225
query44
query45 201 211 186 186
query46 882 982 605 605
query47 2115 2136 2057 2057
query48 321 327 228 228
query49 635 486 387 387
query50 693 272 222 222
query51 4093 4130 4122 4122
query52 291 296 280 280
query53 300 336 287 287
query54 308 276 267 267
query55 93 88 90 88
query56 320 318 314 314
query57 1384 1346 1282 1282
query58 289 276 279 276
query59 1356 1400 1301 1301
query60 345 338 321 321
query61 155 140 140 140
query62 608 598 535 535
query63 312 278 280 278
query64 5034 1263 983 983
query65
query66 1449 458 358 358
query67 16313 16591 16377 16377
query68
query69 388 315 284 284
query70 998 966 974 966
query71 340 312 309 309
query72 2760 2704 2451 2451
query73 544 552 321 321
query74 10000 9951 9797 9797
query75 2841 2758 2476 2476
query76 2280 1047 687 687
query77 366 387 315 315
query78 11194 11267 10665 10665
query79 2974 841 598 598
query80 1726 621 540 540
query81 601 282 251 251
query82 1001 162 121 121
query83 335 267 251 251
query84 298 115 102 102
query85 896 475 428 428
query86 503 295 273 273
query87 3129 3143 3012 3012
query88 3563 2698 2654 2654
query89 423 387 341 341
query90 2245 179 174 174
query91 161 163 137 137
query92 93 78 70 70
query93 2164 808 492 492
query94 668 321 278 278
query95 592 343 314 314
query96 644 514 230 230
query97 2486 2509 2430 2430
query98 246 221 218 218
query99 945 991 927 927
Total cold run time: 238177 ms
Total hot run time: 153077 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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4052497036
# BE Regression && UT Coverage Report
Increment line coverage `100% (0/0)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/61250_6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/61250_6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.46% (26176/36628) |
| Line Coverage | 54.20% (274414/506264) |
| Region Coverage | 51.16% (226757/443237) |
| Branch Coverage | 52.73% (97842/185564) |
--
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "csun5285 (via GitHub)" <gi...@apache.org>.
csun5285 commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4044389605
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4052987336
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4052987463
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4044389483
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #61250:
URL: https://github.com/apache/doris/pull/61250#issuecomment-4045692932
# BE UT Coverage Report
Increment line coverage `0.00% (0/5)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c_6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c_6d0b9f0bed84c550be292776f3e8a4cf6e65bb1c/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.64% (19684/37397) |
| Line Coverage | 36.21% (183672/507223) |
| Region Coverage | 32.32% (141785/438652) |
| Branch Coverage | 33.48% (61877/184818) |
--
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] [refactor](be) Move PackedUInt128 to packed_int128.h [doris]
Posted by "yiguolei (via GitHub)" <gi...@apache.org>.
yiguolei merged PR #61250:
URL: https://github.com/apache/doris/pull/61250
--
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