Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-20 Thread via GitHub


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

   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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-20 Thread via GitHub


morrySnow merged PR #51058:
URL: https://github.com/apache/doris/pull/51058


-- 
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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-20 Thread via GitHub


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

   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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


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

   
   
   ClickBench: Total hot run time: 28.99 s
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
   ClickBench test result on commit c96eda51c349c8c5d7526348b6f1497576d22e92, 
data reload: false
   
   query1   0.040.040.04
   query2   0.120.110.10
   query3   0.260.200.19
   query4   1.590.190.19
   query5   0.450.450.44
   query6   1.510.660.68
   query7   0.020.020.02
   query8   0.040.030.04
   query9   0.600.540.53
   query10  0.590.590.59
   query11  0.150.120.12
   query12  0.140.120.12
   query13  0.610.600.60
   query14  0.800.800.80
   query15  0.860.850.86
   query16  0.380.390.38
   query17  1.001.041.04
   query18  0.220.210.20
   query19  1.891.801.82
   query20  0.010.010.01
   query21  15.40   0.880.53
   query22  0.761.160.68
   query23  14.95   1.340.61
   query24  6.732.210.77
   query25  0.510.240.06
   query26  0.610.160.14
   query27  0.060.050.05
   query28  9.340.820.44
   query29  12.54   3.973.33
   query30  0.250.090.06
   query31  3.120.600.38
   query32  3.230.530.47
   query33  3.113.043.16
   query34  15.60   5.054.48
   query35  4.504.494.47
   query36  0.670.500.47
   query37  0.080.060.07
   query38  0.060.040.03
   query39  0.040.030.02
   query40  0.160.140.13
   query41  0.090.030.02
   query42  0.040.030.02
   query43  0.040.030.02
   Total cold run time: 103.17 s
   Total hot run time: 28.99 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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


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

   
   
   TPC-DS: Total hot run time: 185502 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 c96eda51c349c8c5d7526348b6f1497576d22e92, 
data reload: false
   
   query1   1009472 511 472
   query2   6577182418081808
   query3   6767219 225 219
   query4   26700   23701   22967   22967
   query5   4352622 448 448
   query6   296 200 194 194
   query7   4618492 294 294
   query8   288 232 227 227
   query9   8611262726542627
   query10  472 337 282 282
   query11  15416   15022   14890   14890
   query12  154 106 98  98
   query13  1639505 402 402
   query14  8647608260896082
   query15  205 191 173 173
   query16  7136630 464 464
   query17  950 735 593 593
   query18  1979402 301 301
   query19  195 195 179 179
   query20  121 113 122 113
   query21  221 125 110 110
   query22  4203412241124112
   query23  34015   33007   33127   33007
   query24  8414241724002400
   query25  560 500 416 416
   query26  1237270 157 157
   query27  2754510 341 341
   query28  4313212720962096
   query29  785 625 427 427
   query30  288 216 185 185
   query31  931 849 763 763
   query32  85  62  62  62
   query33  564 364 312 312
   query34  784 832 535 535
   query35  786 794 731 731
   query36  954 998 860 860
   query37  116 102 75  75
   query38  4162406840114011
   query39  1550141014061406
   query40  212 121 104 104
   query41  57  58  65  58
   query42  137 114 107 107
   query43  499 518 490 490
   query44  1309831 820 820
   query45  176 177 169 169
   query46  832 1019633 633
   query47  1755180917201720
   query48  386 423 317 317
   query49  771 508 424 424
   query50  653 673 413 413
   query51  4108419540444044
   query52  119 109 98  98
   query53  221 250 190 190
   query54  594 575 502 502
   query55  83  87  83  83
   query56  328 293 277 277
   query57  1118115510761076
   query58  262 255 253 253
   query59  2537262526252625
   query60  320 324 319 319
   query61  129 118 121 118
   query62  795 736 658 658
   query63  236 190 191 190
   query64  4365991 653 653
   query65  4284416942654169
   query66  1152408 321 321
   query67  15971   15505   15456   15456
   query68  8489876 528 528
   query69  488 316 266 266
   query70  1173112110721072
   query71  478 327 313 313
   query72  5405471348314713
   query73  722 628 355 355
   query74  8873888388678867
   query75  4064317726672667
   query76  37251179741 741
   query77  785 392 293 293
   query78  10034   10239   93559355
   query79  2015819 569 569
   query80  624 548 442 442
   query81  469 259 230 230
   query82  464 126 94  94
   query83  289 263 234 234
   query84  295 111 82  82
   query85  814 366 313 313
   query86  343 330 279 279
   query87  4366437844134378
   query88  2953233623242324
   query89  399 319 281 281
   query90  1950208 204 204
   query91  139 137 114 114
   query92  77  59  59  59
   query93  1390927 586 586
   query94  674 395 320 320
   query95  370 289 276 276
   query96  494 583 282 282
   query97  2730272226792679
   query98  227 208 220 208
   query99  1416136912521252
   Total cold run time: 272997 ms
   Total hot run time: 185502 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 t

Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


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

   
   
   TPC-H: Total hot run time: 33687 ms
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit c96eda51c349c8c5d7526348b6f1497576d22e92, 
data reload: false
   
   -- Round 1 --
   q1   26447   502949914991
   q2   2094287 176 176
   q3   10404   1235708 708
   q4   10236   1012527 527
   q5   7495240923122312
   q6   179 163 130 130
   q7   908 730 614 614
   q8   9316128010911091
   q9   6749506250055005
   q10  6820231018841884
   q11  478 283 285 283
   q12  357 359 217 217
   q13  17789   361430963096
   q14  243 228 219 219
   q15  546 491 502 491
   q16  441 428 378 378
   q17  574 874 360 360
   q18  7914714471807144
   q19  1237961 556 556
   q20  330 330 232 232
   q21  3724253023002300
   q22  10471008973 973
   Total cold run time: 115328 ms
   Total hot run time: 33687 ms
   
   - Round 2, with runtime_filter_mode=off -
   q1   5087506950815069
   q2   239 325 225 225
   q3   2178265322682268
   q4   1354181013981398
   q5   4448448944414441
   q6   222 172 137 137
   q7   2059191717571757
   q8   2593246026162460
   q9   7048722172177217
   q10  3026315227412741
   q11  577 515 502 502
   q12  685 788 612 612
   q13  3523395932323232
   q14  271 303 260 260
   q15  542 500 500 500
   q16  458 489 430 430
   q17  1118152213921392
   q18  7768756674307430
   q19  815 748 844 748
   q20  1974198618901890
   q21  4871435743404340
   q22  10411032999 999
   Total cold run time: 51897 ms
   Total hot run time: 50048 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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


morrySnow commented on PR #51058:
URL: https://github.com/apache/doris/pull/51058#issuecomment-2892890184

   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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



Re: [PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


Thearas commented on PR #51058:
URL: https://github.com/apache/doris/pull/51058#issuecomment-2892884038

   
   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-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org



[PR] [fix](metadata) replace view will break view's metadata [doris]

2025-05-19 Thread via GitHub


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

   ### What problem does this PR solve?
   
   Related PR: #40715
   
   Problem Summary:
   
   this problem intro by #40715, when do replace view, full schema do not be 
set correctly. So view's schema will lost after replace.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test 
   - [x] 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:
   - [x] No.
   - [ ] Yes. 
   
   - Does this need documentation?
   - [x] 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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org