You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "yx-keith (via GitHub)" <gi...@apache.org> on 2025/10/14 07:42:15 UTC
[PR] [Enhancement](nereids) remove class PartitionNames [doris]
yx-keith opened a new pull request, #56947:
URL: https://github.com/apache/doris/pull/56947
### 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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3400776698
<details>
<summary>ClickBench: <b>Total hot run time: 30.34 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 2120d18fff7e4a8451eda40c49e6c79fc7483a4b, data reload: false
query1 0.06 0.05 0.06
query2 0.09 0.05 0.07
query3 0.24 0.08 0.08
query4 1.60 0.12 0.12
query5 0.27 0.26 0.26
query6 1.18 0.65 0.66
query7 0.03 0.03 0.03
query8 0.05 0.04 0.05
query9 0.63 0.53 0.55
query10 0.58 0.58 0.58
query11 0.16 0.12 0.12
query12 0.16 0.12 0.12
query13 0.63 0.62 0.62
query14 1.04 1.03 1.04
query15 0.88 0.88 0.89
query16 0.44 0.39 0.42
query17 1.07 1.06 1.08
query18 0.22 0.20 0.21
query19 1.97 1.92 1.85
query20 0.02 0.02 0.02
query21 15.41 0.92 0.56
query22 0.76 1.20 0.75
query23 14.78 1.39 0.64
query24 6.77 1.84 0.61
query25 0.47 0.18 0.06
query26 0.48 0.16 0.12
query27 0.07 0.06 0.05
query28 9.65 1.38 0.94
query29 12.58 4.03 3.28
query30 0.27 0.14 0.11
query31 2.84 0.61 0.38
query32 3.25 0.57 0.48
query33 3.25 3.12 3.18
query34 16.11 5.51 4.83
query35 4.92 4.89 4.86
query36 0.73 0.53 0.51
query37 0.10 0.08 0.07
query38 0.06 0.05 0.04
query39 0.03 0.03 0.04
query40 0.19 0.15 0.14
query41 0.09 0.03 0.02
query42 0.04 0.03 0.03
query43 0.04 0.04 0.03
Total cold run time: 104.21 s
Total hot run time: 30.34 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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3401503115
# FE Regression Coverage Report
Increment line coverage ` 67.50% (54/80)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/report/index.html)
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "yx-keith (via GitHub)" <gi...@apache.org>.
yx-keith commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3401540848
run cloud_p0
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3404233644
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3400510475
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3402734385
# FE Regression Coverage Report
Increment line coverage ` 65.00% (52/80)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/report/index.html)
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3402225409
# FE Regression Coverage Report
Increment line coverage ` 65.00% (52/80)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/56947_2120d18fff7e4a8451eda40c49e6c79fc7483a4b_merge_fe/report/index.html)
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "yx-keith (via GitHub)" <gi...@apache.org>.
yx-keith commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3402145390
run cloud_p0
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "yx-keith (via GitHub)" <gi...@apache.org>.
yx-keith commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3400572563
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3400757446
<details>
<summary>TPC-DS: <b>Total hot run time: 189695 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 2120d18fff7e4a8451eda40c49e6c79fc7483a4b, data reload: false
query1 1067 452 405 405
query2 6575 1690 1707 1690
query3 6763 233 226 226
query4 26327 23722 23374 23374
query5 5354 639 482 482
query6 358 233 250 233
query7 4645 496 298 298
query8 293 255 276 255
query9 8716 2566 2554 2554
query10 543 358 297 297
query11 15903 15012 15138 15012
query12 185 117 115 115
query13 1674 545 428 428
query14 11642 9245 9177 9177
query15 207 189 178 178
query16 8487 720 571 571
query17 1287 777 658 658
query18 2056 452 361 361
query19 226 219 172 172
query20 132 128 130 128
query21 242 138 116 116
query22 4655 4808 4560 4560
query23 34582 33495 34065 33495
query24 8334 2505 2549 2505
query25 651 572 528 528
query26 1268 294 168 168
query27 2792 531 382 382
query28 4375 2223 2202 2202
query29 977 639 582 582
query30 308 225 187 187
query31 887 795 725 725
query32 84 70 67 67
query33 577 393 312 312
query34 813 856 528 528
query35 798 815 751 751
query36 982 1010 907 907
query37 117 106 84 84
query38 3515 3548 3463 3463
query39 1474 1407 1414 1407
query40 219 124 117 117
query41 61 58 58 58
query42 126 108 121 108
query43 496 491 522 491
query44 1339 833 824 824
query45 181 180 166 166
query46 835 1003 647 647
query47 1801 1855 1738 1738
query48 387 408 311 311
query49 767 527 405 405
query50 644 695 410 410
query51 3837 3949 3879 3879
query52 106 105 103 103
query53 236 268 193 193
query54 615 589 526 526
query55 87 89 89 89
query56 330 311 307 307
query57 1172 1185 1139 1139
query58 277 272 281 272
query59 2580 2711 2518 2518
query60 348 346 323 323
query61 158 156 156 156
query62 812 740 719 719
query63 237 200 195 195
query64 4458 1185 865 865
query65 4074 3955 3949 3949
query66 1067 434 344 344
query67 15440 15232 15142 15142
query68 9321 878 590 590
query69 497 314 286 286
query70 1358 1270 1362 1270
query71 497 342 319 319
query72 5962 4798 4755 4755
query73 700 571 358 358
query74 8976 9093 8623 8623
query75 4444 3327 2824 2824
query76 4515 1162 747 747
query77 1042 396 314 314
query78 9681 9850 8822 8822
query79 2480 853 602 602
query80 689 642 517 517
query81 484 271 222 222
query82 437 232 132 132
query83 288 268 255 255
query84 300 118 95 95
query85 876 461 435 435
query86 345 299 311 299
query87 3712 3754 3744 3744
query88 2937 2267 2251 2251
query89 420 316 296 296
query90 2044 226 222 222
query91 159 163 146 146
query92 85 76 66 66
query93 1921 1011 638 638
query94 714 454 317 317
query95 402 317 316 316
query96 496 577 288 288
query97 2956 2973 2875 2875
query98 238 207 212 207
query99 1468 1401 1287 1287
Total cold run time: 283005 ms
Total hot run time: 189695 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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "starocean999 (via GitHub)" <gi...@apache.org>.
starocean999 merged PR #56947:
URL: https://github.com/apache/doris/pull/56947
--
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3404233690
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] [Enhancement](nereids) remove class PartitionNames [doris]
Posted by "yx-keith (via GitHub)" <gi...@apache.org>.
yx-keith commented on PR #56947:
URL: https://github.com/apache/doris/pull/56947#issuecomment-3400510709
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