Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-16 Thread via GitHub


wangbo merged PR #51703:
URL: https://github.com/apache/doris/pull/51703


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


hello-stephen commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969908674

   # BE Regression && UT Coverage Report
   Increment line coverage `100.00% (1/1)` :tada:
   
   [Increment coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12_merge/increment_report/index.html)
   [Complete coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12_merge/report/index.html)
   | Category  | Coverage   |
   |---||
   | Function Coverage | 79.75% (20974/26301) |
   | Line Coverage | 72.63% (216604/298211) |
   | Region Coverage   | 70.89% (127689/180135) |
   | Branch Coverage   | 64.55% (66103/102398) |


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


doris-robot commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969807513

   
   
   ClickBench: Total hot run time: 30.03 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit c39fece065c7d035239405bc2a11dc419f652b12, 
data reload: false
   
   query1   0.040.040.03
   query2   0.080.040.04
   query3   0.240.070.06
   query4   1.600.100.11
   query5   0.480.440.47
   query6   1.170.710.68
   query7   0.030.030.01
   query8   0.060.040.04
   query9   0.620.550.54
   query10  0.610.630.61
   query11  0.190.120.13
   query12  0.170.130.13
   query13  0.630.610.61
   query14  0.810.840.85
   query15  0.920.890.91
   query16  0.410.410.41
   query17  1.091.091.13
   query18  0.250.230.23
   query19  2.032.031.89
   query20  0.020.010.02
   query21  15.43   1.040.64
   query22  0.781.250.78
   query23  14.75   1.510.72
   query24  7.020.890.36
   query25  0.380.160.17
   query26  0.740.200.18
   query27  0.060.050.05
   query28  8.821.020.48
   query29  12.56   4.543.80
   query30  0.260.100.07
   query31  2.840.670.42
   query32  3.250.580.49
   query33  3.083.183.20
   query34  15.79   5.204.49
   query35  4.594.564.63
   query36  0.690.520.51
   query37  0.100.070.07
   query38  0.060.050.04
   query39  0.030.030.02
   query40  0.180.140.14
   query41  0.100.030.03
   query42  0.040.030.03
   query43  0.040.040.04
   Total cold run time: 103.04 s
   Total hot run time: 30.03 s
   ```
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


doris-robot commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969793027

   
   
   TPC-DS: Total hot run time: 194643 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
   TPC-DS sf100 test result on commit c39fece065c7d035239405bc2a11dc419f652b12, 
data reload: false
   
   query1   1046436 410 410
   query2   6552210321052103
   query3   6755265 260 260
   query4   27286   23760   23324   23324
   query5   4403712 531 531
   query6   337 267 233 233
   query7   4648559 361 361
   query8   298 259 249 249
   query9   8675321932623219
   query10  529 376 315 315
   query11  15564   15160   14915   14915
   query12  176 124 121 121
   query13  1688635 534 534
   query14  8921651866276518
   query15  222 228 200 200
   query16  7372661 558 558
   query17  1239807 632 632
   query18  2067428 325 325
   query19  223 224 198 198
   query20  130 132 123 123
   query21  222 136 120 120
   query22  4304444142784278
   query23  34539   33402   33508   33402
   query24  8548257625512551
   query25  549 482 447 447
   query26  1237297 175 175
   query27  2687547 397 397
   query28  4343260825712571
   query29  846 625 503 503
   query30  306 236 212 212
   query31  969 916 849 849
   query32  81  76  75  75
   query33  590 442 340 340
   query34  885 941 623 623
   query35  857 850 770 770
   query36  1081112210331033
   query37  130 115 87  87
   query38  4271439642284228
   query39  1573144414901444
   query40  229 136 124 124
   query41  76  75  68  68
   query42  146 132 134 132
   query43  582 591 554 554
   query44  1529986 1017986
   query45  194 192 181 181
   query46  963 1127719 719
   query47  1850187417801780
   query48  464 475 369 369
   query49  783 541 450 450
   query50  764 735 467 467
   query51  4261428842424242
   query52  131 128 123 123
   query53  265 282 212 212
   query54  698 675 611 611
   query55  105 93  98  93
   query56  356 360 327 327
   query57  1242121811801180
   query58  317 313 297 297
   query59  2920306529122912
   query60  378 378 353 353
   query61  145 157 146 146
   query62  817 766 683 683
   query63  269 226 217 217
   query64  44181114783 783
   query65  4340421142204211
   query66  1170451 340 340
   query67  16606   16350   16000   16000
   query68  81801029696 696
   query69  520 370 309 309
   query70  1386122412881224
   query71  526 378 358 358
   query72  5824492849164916
   query73  752 674 430 430
   query74  9105893088318831
   query75  4005328128242824
   query76  37861236836 836
   query77  814 421 339 339
   query78  10306   10366   95489548
   query79  2340947 684 684
   query80  699 647 532 532
   query81  489 265 245 245
   query82  437 137 112 112
   query83  309 291 277 277
   query84  309 122 109 109
   query85  821 464 370 370
   query86  364 341 334 334
   query87  4614469344934493
   query88  3832273526722672
   query89  420 351 318 318
   query90  1936235 224 224
   query91  169 176 133 133
   query92  77  68  72  68
   query93  15771121719 719
   query94  703 454 322 322
   query95  430 342 337 337
   query96  604 636 319 319
   query97  2824285227292729
   query98  257 236 218 218
   query99  1480141713231323
   Total cold run time: 282062 ms
   Total hot run time: 194643 ms
   ```
   
   


-- 
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

Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


hello-stephen commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969766146

   # BE Regression && UT Coverage Report
   Increment line coverage `100.00% (1/1)` :tada:
   
   [Increment coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12_merge/increment_report/index.html)
   [Complete coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12_merge/report/index.html)
   | Category  | Coverage   |
   |---||
   | Function Coverage | 79.75% (20974/26301) |
   | Line Coverage | 72.63% (216604/298211) |
   | Region Coverage   | 70.89% (127689/180135) |
   | Branch Coverage   | 64.55% (66103/102398) |


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


doris-robot commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969759693

   
   
   TPC-H: Total hot run time: 36198 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit c39fece065c7d035239405bc2a11dc419f652b12, 
data reload: false
   
   -- Round 1 --
   q1   17682   509850285028
   q2   1968317 217 217
   q3   10223   1339787 787
   q4   10241   1057596 596
   q5   7577272424312431
   q6   195 176 140 140
   q7   962 778 644 644
   q8   9355144612891289
   q9   6990521951185118
   q10  6871235919291929
   q11  541 329 322 322
   q12  387 392 249 249
   q13  17788   380532443244
   q14  250 253 238 238
   q15  560 484 487 484
   q16  502 486 409 409
   q17  732 888 451 451
   q18  8244734672327232
   q19  1223990 618 618
   q20  412 405 260 260
   q21  4673406134803480
   q22  1149109110321032
   Total cold run time: 108525 ms
   Total hot run time: 36198 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   5064506550815065
   q2   269 342 226 226
   q3   2263274323752375
   q4   1514190714751475
   q5   4579450344384438
   q6   233 171 129 129
   q7   2029196017691769
   q8   2622255526862555
   q9   7199722273757222
   q10  3131325927842784
   q11  613 550 523 523
   q12  759 809 694 694
   q13  3677399535193519
   q14  298 302 293 293
   q15  547 477 495 477
   q16  486 534 475 475
   q17  1239156514091409
   q18  7866761075087508
   q19  946 996 1118996
   q20  2049206319681968
   q21  5381466346754663
   q22  1135111710501050
   Total cold run time: 53899 ms
   Total hot run time: 51613 ms
   ```
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


hello-stephen commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969684062

   # BE UT Coverage Report
   Increment line coverage `0.00% (0/1)` :tada:
   
   [Increment coverage 
report](http://coverage.selectdb-in.cc/coverage/c39fece065c7d035239405bc2a11dc419f652b12_c39fece065c7d035239405bc2a11dc419f652b12/increment_report/index.html)
   [Complete coverage 
report](http://coverage.selectdb-in.cc/coverage/c39fece065c7d035239405bc2a11dc419f652b12_c39fece065c7d035239405bc2a11dc419f652b12/report/index.html)
   | Category  | Coverage   |
   |---||
   | Function Coverage | 56.40% (15072/26724) |
   | Line Coverage | 45.16% (134747/298369) |
   | Region Coverage   | 44.28% (67824/153162) |
   | Branch Coverage   | 38.84% (34798/89592) |


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


hello-stephen commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969653545

   # FE UT Coverage Report
   Increment line coverage `  50.00% (2/4)` :tada:
   [Increment coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12/fe_increment_report/index.html)
   [Complete coverage 
report](http://coverage.selectdb-in.cc/coverage/51703_c39fece065c7d035239405bc2a11dc419f652b12/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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


github-actions[bot] commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969482161

   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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


github-actions[bot] commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969482054

   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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


wangbo commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969386297

   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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



[PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


wangbo opened a new pull request, #51703:
URL: https://github.com/apache/doris/pull/51703

   ### What problem does this PR solve?
   
   Thoroughly fix for #51702
   Issue Number: close #xxx
   
   Related PR: #xxx
   
   Problem Summary:
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test 
   - [ ] 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 
   
   - Behavior changed:
   - [ ] No.
   - [ ] Yes. 
   
   - Does this need documentation?
   - [ ] No.
   - [ ] Yes. 
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label 
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]



Re: [PR] [Fix]Fix FE send duplicate wg id to BE [doris]

2025-06-13 Thread via GitHub


hello-stephen commented on PR #51703:
URL: https://github.com/apache/doris/pull/51703#issuecomment-2969385273

   
   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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]