Skip to content

[chore](alter job) Fix finished alter job print too much warning log#55132

Merged
BiteTheDDDDt merged 1 commit intoapache:masterfrom
xy720:fix-alter-job-log
Aug 22, 2025
Merged

[chore](alter job) Fix finished alter job print too much warning log#55132
BiteTheDDDDt merged 1 commit intoapache:masterfrom
xy720:fix-alter-job-log

Conversation

@xy720
Copy link
Member

@xy720 xy720 commented Aug 21, 2025 •
edited
Loading

What problem does this PR solve?

Related PR: #54113

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

@Thearas
Copy link
Contributor

Thearas commented Aug 21, 2025

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See 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?

@xy720
Copy link
Member Author

xy720 commented Aug 21, 2025

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Aug 21, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 21, 2025

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 21, 2025

PR approved by anyone and no changes requested.

@doris-robot
Copy link

doris-robot commented Aug 21, 2025

TPC-H: Total hot run time: 33507 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 54437013f6efd1ff7d497a6c9d2a15a5ebf4ed64, data reload: false

------ Round 1 ----------------------------------
q1	17616	5172	5003	5003
q2	1912	288	191	191
q3	10291	1233	692	692
q4	10220	1009	520	520
q5	7530	2399	2235	2235
q6	178	160	134	134
q7	910	727	610	610
q8	9317	1292	1055	1055
q9	6858	5107	5167	5107
q10	6935	2376	1924	1924
q11	488	296	265	265
q12	350	343	210	210
q13	17796	3615	2990	2990
q14	230	237	212	212
q15	546	484	474	474
q16	432	412	367	367
q17	612	836	365	365
q18	7487	6996	6951	6951
q19	1351	953	555	555
q20	343	326	212	212
q21	3902	2469	2929	2469
q22	1065	1021	966	966
Total cold run time: 106369 ms
Total hot run time: 33507 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5248	5092	5095	5092
q2	239	327	218	218
q3	2183	2695	2275	2275
q4	1317	1721	1320	1320
q5	4200	4367	4528	4367
q6	227	172	136	136
q7	2021	1993	1794	1794
q8	2602	2640	2568	2568
q9	7255	7241	7224	7224
q10	3138	3277	2903	2903
q11	617	504	507	504
q12	685	763	602	602
q13	3426	4218	3167	3167
q14	278	293	266	266
q15	498	486	464	464
q16	425	503	441	441
q17	1182	1606	1402	1402
q18	7892	7769	7534	7534
q19	865	900	936	900
q20	1994	2027	1960	1960
q21	4855	4329	4255	4255
q22	1076	1035	964	964
Total cold run time: 52223 ms
Total hot run time: 50356 ms

@doris-robot
Copy link

doris-robot commented Aug 21, 2025

TPC-DS: Total hot run time: 184109 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 54437013f6efd1ff7d497a6c9d2a15a5ebf4ed64, data reload: false

query1	995	379	392	379
query2	6512	1704	1689	1689
query3	6754	216	223	216
query4	26144	23556	22778	22778
query5	4411	617	475	475
query6	317	217	214	214
query7	4632	528	283	283
query8	275	233	229	229
query9	8597	2898	2903	2898
query10	482	320	296	296
query11	16019	14880	14865	14865
query12	166	114	112	112
query13	1663	542	425	425
query14	8856	5766	5689	5689
query15	202	177	169	169
query16	7250	613	478	478
query17	1219	727	594	594
query18	2009	405	319	319
query19	190	183	167	167
query20	135	123	114	114
query21	216	131	110	110
query22	4101	4110	3981	3981
query23	33553	33319	32846	32846
query24	8129	2368	2372	2368
query25	529	455	401	401
query26	1228	273	206	206
query27	2695	499	338	338
query28	4352	2245	2216	2216
query29	767	569	465	465
query30	286	218	190	190
query31	877	807	697	697
query32	83	72	75	72
query33	566	374	345	345
query34	802	838	516	516
query35	803	833	720	720
query36	991	994	909	909
query37	124	115	85	85
query38	3976	4033	3929	3929
query39	1480	1427	1384	1384
query40	208	128	119	119
query41	61	75	54	54
query42	118	109	107	107
query43	515	470	495	470
query44	1361	842	861	842
query45	174	172	162	162
query46	857	1005	629	629
query47	1721	1776	1730	1730
query48	386	442	319	319
query49	711	489	383	383
query50	638	658	421	421
query51	4084	4095	4028	4028
query52	110	104	102	102
query53	238	267	191	191
query54	587	611	516	516
query55	86	83	82	82
query56	316	330	301	301
query57	1158	1190	1102	1102
query58	279	272	261	261
query59	2691	2770	2705	2705
query60	344	342	323	323
query61	129	119	133	119
query62	794	726	676	676
query63	227	191	184	184
query64	4365	1016	744	744
query65	4289	4205	4230	4205
query66	1207	416	323	323
query67	15247	15151	15178	15151
query68	8219	921	574	574
query69	480	318	281	281
query70	1244	1158	1149	1149
query71	466	325	315	315
query72	5681	4651	4775	4651
query73	732	638	355	355
query74	9028	9064	8735	8735
query75	3900	3059	2667	2667
query76	3761	1141	743	743
query77	783	396	315	315
query78	9537	9508	8837	8837
query79	4098	835	589	589
query80	643	548	478	478
query81	482	258	218	218
query82	492	144	194	144
query83	283	248	240	240
query84	290	111	82	82
query85	776	361	332	332
query86	341	306	301	301
query87	4227	4242	4166	4166
query88	3064	2204	2211	2204
query89	443	319	294	294
query90	1975	302	223	223
query91	148	144	109	109
query92	91	70	64	64
query93	2099	980	641	641
query94	678	435	315	315
query95	404	320	304	304
query96	494	572	281	281
query97	2607	2656	2581	2581
query98	227	216	208	208
query99	1459	1437	1297	1297
Total cold run time: 274439 ms
Total hot run time: 184109 ms

@doris-robot
Copy link

doris-robot commented Aug 21, 2025

ClickBench: Total hot run time: 32.56 s 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 54437013f6efd1ff7d497a6c9d2a15a5ebf4ed64, data reload: false

query1	0.04	0.04	0.04
query2	0.08	0.05	0.04
query3	0.24	0.07	0.07
query4	1.63	0.11	0.12
query5	0.43	0.42	0.40
query6	1.15	0.64	0.64
query7	0.02	0.02	0.02
query8	0.04	0.03	0.04
query9	0.59	0.54	0.52
query10	0.57	0.57	0.59
query11	0.17	0.11	0.11
query12	0.16	0.11	0.11
query13	0.62	0.60	0.60
query14	0.81	0.81	0.85
query15	0.87	0.84	0.84
query16	0.37	0.38	0.39
query17	1.08	1.05	1.06
query18	0.22	0.19	0.20
query19	1.91	1.82	1.83
query20	0.01	0.01	0.01
query21	15.38	0.96	0.60
query22	0.79	1.28	0.82
query23	14.74	1.36	0.66
query24	6.58	2.17	0.41
query25	0.29	0.24	0.18
query26	0.70	0.15	0.13
query27	0.06	0.05	0.05
query28	9.10	0.93	0.44
query29	12.59	3.92	3.26
query30	3.14	3.02	3.00
query31	2.82	0.60	0.38
query32	3.23	0.54	0.49
query33	3.08	3.12	3.13
query34	16.11	5.43	4.91
query35	4.92	4.92	5.01
query36	0.69	0.51	0.49
query37	0.10	0.07	0.08
query38	0.06	0.04	0.04
query39	0.03	0.03	0.02
query40	0.18	0.15	0.14
query41	0.08	0.03	0.02
query42	0.04	0.03	0.03
query43	0.04	0.03	0.02
Total cold run time: 105.76 s
Total hot run time: 32.56 s

@hello-stephen
Copy link
Contributor

hello-stephen commented Aug 21, 2025

FE UT Coverage Report

Increment line coverage 100.00% (5/5) 🎉
Increment coverage report
Complete coverage report

@BiteTheDDDDt BiteTheDDDDt merged commit 654467a into apache:master Aug 22, 2025
32 of 33 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 22, 2025
@xy720
[chore](alter job) Fix finished alter job print too much warning log (#…
a21ca4c 
…55132)

Related PR: #54113
@github-actions github-actions bot mentioned this pull request Aug 22, 2025
Merged
github-actions bot pushed a commit that referenced this pull request Aug 22, 2025
@xy720
[chore](alter job) Fix finished alter job print too much warning log (#…
534f36c 
…55132)

Related PR: #54113
@github-actions github-actions bot mentioned this pull request Aug 22, 2025
Merged
dataroaring pushed a commit that referenced this pull request Aug 24, 2025
@github-actions @xy720
branch-3.0: [chore](alter job) Fix finished alter job print too much …
72ebf34 
…warning log #55132 (#55154)

Cherry-picked from #55132

Co-authored-by: xy720 <22125576+xy720@users.noreply.github.com>
morrySnow pushed a commit that referenced this pull request Aug 24, 2025
@github-actions @xy720
branch-3.1: [chore](alter job) Fix finished alter job print too much …
ed844fb 
…warning log #55132 (#55156)

Cherry-picked from #55132

Co-authored-by: xy720 <22125576+xy720@users.noreply.github.com>
@gavinchou gavinchou mentioned this pull request Sep 1, 2025
Open
@xy720 xy720 mentioned this pull request Sep 4, 2025
Merged
16 tasks
yiguolei pushed a commit that referenced this pull request Sep 5, 2025
@xy720
branch-2.1: [bug](group commit) Fix group commit blocked after schema…
ace284b 
… change throw exception (#55684)

### What problem does this PR solve?

pick #54113 #55132

### Check List (For Author)

- Test <!-- At least one of them must be included. -->
    - [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 <!-- Add your reason?  -->

- Behavior changed:
    - [x] No.
    - [ ] Yes. <!-- Explain the behavior change -->

- Does this need documentation?
    - [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
apache/doris-website#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 -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BiteTheDDDDt BiteTheDDDDt BiteTheDDDDt approved these changes

@zclllyybb zclllyybb zclllyybb approved these changes

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. dev/2.1.12-merged dev/3.0.8-merged dev/3.1.0-merged reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@xy720 @Thearas @doris-robot @hello-stephen @BiteTheDDDDt @zclllyybb @yiguolei @dataroaring @morrySnow

Comments