Skip to content

[fix](ut)Fix StmtExecutorTest #50322

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 23, 2025
Merged

[fix](ut)Fix StmtExecutorTest #50322

merged 1 commit into from
Apr 23, 2025

Conversation

Jibing-Li
Copy link
Contributor

@Jibing-Li Jibing-Li commented Apr 23, 2025 •
edited by morrySnow
Loading

What problem does this PR solve?

Fix StmtExecutorTest
Issue Number: close #xxx

Related PR: #49984

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 Apr 23, 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?

@Jibing-Li Jibing-Li marked this pull request as ready for review April 23, 2025 04:21
@Jibing-Li
Copy link
Contributor Author

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Apr 23, 2025
@github-actions GitHub Actions
Copy link
Contributor

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

@github-actions GitHub Actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	26010	4988	4982	4982
q2	2075	292	183	183
q3	10375	1260	687	687
q4	10222	1007	522	522
q5	7531	2361	2295	2295
q6	180	160	130	130
q7	901	730	599	599
q8	9323	1298	1001	1001
q9	6946	5067	5080	5067
q10	6805	2344	1871	1871
q11	493	299	276	276
q12	350	360	218	218
q13	17775	3679	3089	3089
q14	225	221	207	207
q15	528	484	496	484
q16	434	450	394	394
q17	583	855	391	391
q18	7493	7225	7199	7199
q19	1219	944	545	545
q20	356	336	226	226
q21	3975	2708	2408	2408
q22	1051	1018	974	974
Total cold run time: 114850 ms
Total hot run time: 33748 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5164	5031	5031	5031
q2	247	336	232	232
q3	2129	2672	2282	2282
q4	1443	1838	1404	1404
q5	4452	4412	4424	4412
q6	218	169	130	130
q7	1936	1921	1747	1747
q8	2545	2474	2438	2438
q9	7200	7140	7124	7124
q10	3027	3183	2751	2751
q11	576	484	487	484
q12	672	743	602	602
q13	3556	3883	3301	3301
q14	285	307	263	263
q15	513	481	470	470
q16	450	499	462	462
q17	1156	1568	1378	1378
q18	7573	7615	7359	7359
q19	760	804	991	804
q20	1969	2074	1887	1887
q21	5335	4678	4737	4678
q22	1070	1022	992	992
Total cold run time: 52276 ms
Total hot run time: 50231 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 185929 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 985d5308625857308bc8f6b11225786d1e2179c8, data reload: false

query1	1028	472	480	472
query2	6567	1902	1854	1854
query3	6748	225	213	213
query4	26256	23688	23426	23426
query5	4354	625	463	463
query6	302	218	216	216
query7	4624	494	280	280
query8	309	244	241	241
query9	8617	2548	2579	2548
query10	474	325	258	258
query11	15787	15035	14746	14746
query12	166	108	106	106
query13	1648	508	392	392
query14	9427	6116	6035	6035
query15	199	185	192	185
query16	7245	627	471	471
query17	1176	704	561	561
query18	1962	402	298	298
query19	186	178	163	163
query20	117	118	126	118
query21	210	125	105	105
query22	4228	4188	4030	4030
query23	34124	32960	33097	32960
query24	8431	2350	2374	2350
query25	519	476	394	394
query26	1217	276	143	143
query27	2757	493	360	360
query28	4314	2101	2085	2085
query29	762	554	433	433
query30	283	216	187	187
query31	925	870	794	794
query32	76	64	65	64
query33	579	369	305	305
query34	783	832	531	531
query35	774	808	733	733
query36	944	991	883	883
query37	109	101	76	76
query38	4215	4174	4128	4128
query39	1460	1439	1425	1425
query40	207	117	104	104
query41	55	58	53	53
query42	116	98	101	98
query43	496	513	490	490
query44	1273	801	801	801
query45	194	179	176	176
query46	825	1018	620	620
query47	1796	1783	1767	1767
query48	394	403	298	298
query49	778	539	406	406
query50	624	675	417	417
query51	4202	4079	4024	4024
query52	99	101	95	95
query53	218	254	185	185
query54	575	568	482	482
query55	84	78	82	78
query56	299	299	278	278
query57	1129	1152	1093	1093
query58	264	261	250	250
query59	2646	2757	2571	2571
query60	313	317	338	317
query61	135	165	126	126
query62	801	739	656	656
query63	223	187	185	185
query64	4350	1031	690	690
query65	4358	4219	4207	4207
query66	1132	414	304	304
query67	15773	15651	15363	15363
query68	5510	879	501	501
query69	467	302	262	262
query70	1168	1097	1097	1097
query71	412	310	288	288
query72	5524	4704	4800	4704
query73	652	598	344	344
query74	9249	9313	8925	8925
query75	3151	3198	2727	2727
query76	3159	1177	730	730
query77	479	378	289	289
query78	9850	10039	9299	9299
query79	1811	805	556	556
query80	597	514	440	440
query81	478	253	219	219
query82	190	125	97	97
query83	259	248	231	231
query84	251	108	88	88
query85	771	353	314	314
query86	370	302	292	292
query87	4352	4418	4386	4386
query88	2824	2189	2201	2189
query89	378	304	286	286
query90	1961	210	207	207
query91	143	141	113	113
query92	76	59	55	55
query93	1842	916	573	573
query94	701	424	325	325
query95	381	303	294	294
query96	478	564	292	292
query97	3174	3203	3159	3159
query98	249	218	202	202
query99	1328	1427	1279	1279
Total cold run time: 269496 ms
Total hot run time: 185929 ms

@doris-robot
Copy link

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

query1	0.05	0.04	0.02
query2	0.12	0.11	0.11
query3	0.24	0.20	0.20
query4	1.60	0.20	0.19
query5	0.59	0.62	0.59
query6	1.18	0.70	0.73
query7	0.03	0.01	0.02
query8	0.05	0.04	0.04
query9	0.58	0.53	0.53
query10	0.56	0.57	0.56
query11	0.15	0.10	0.11
query12	0.15	0.12	0.12
query13	0.61	0.59	0.60
query14	1.23	1.15	1.16
query15	0.89	0.86	0.87
query16	0.39	0.40	0.38
query17	1.01	1.06	1.02
query18	0.22	0.20	0.20
query19	1.95	1.78	1.84
query20	0.01	0.02	0.01
query21	15.51	0.91	0.58
query22	0.76	1.15	0.93
query23	14.69	1.33	0.68
query24	6.97	1.85	0.50
query25	0.53	0.21	0.07
query26	0.64	0.16	0.13
query27	0.05	0.04	0.05
query28	9.32	0.86	0.44
query29	12.59	4.03	3.35
query30	0.26	0.11	0.06
query31	2.82	0.59	0.39
query32	3.23	0.54	0.46
query33	3.02	3.10	3.04
query34	15.74	5.12	4.55
query35	4.56	4.51	4.52
query36	0.68	0.49	0.49
query37	0.09	0.06	0.07
query38	0.05	0.05	0.04
query39	0.03	0.02	0.03
query40	0.16	0.14	0.13
query41	0.07	0.03	0.02
query42	0.03	0.02	0.02
query43	0.03	0.04	0.03
Total cold run time: 103.44 s
Total hot run time: 29.7 s

@hello-stephen
Copy link
Contributor

skip buildall

@hello-stephen hello-stephen merged commit df664db into apache:master Apr 23, 2025
27 of 28 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 23, 2025
@Jibing-Li
[fix](ut)Fix StmtExecutorTest (#50322)
d252302 
### What problem does this PR solve?
Fix StmtExecutorTest
@github-actions github-actions bot mentioned this pull request Apr 23, 2025
Closed
@Jibing-Li Jibing-Li deleted the fixut branch April 24, 2025 02:07
Jibing-Li added a commit to Jibing-Li/incubator-doris that referenced this pull request Apr 24, 2025
@Jibing-Li
[fix](ut)Fix StmtExecutorTest (apache#50322)
067b377 
Fix StmtExecutorTest
@Jibing-Li Jibing-Li mentioned this pull request Apr 24, 2025
Merged
yiguolei pushed a commit that referenced this pull request Apr 24, 2025
@Jibing-Li
branch-2.1:[fix](ut)Fix StmtExecutorTest (#50322) (#50362)
dd43f13 
backport: #50322
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hello-stephen hello-stephen hello-stephen approved these changes

@morrySnow morrySnow morrySnow approved these changes

Assignees
No one assigned
Labels
approved Indicates a PR has been approved by one committer. dev/2.1.10-merged reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Jibing-Li @Thearas @doris-robot @hello-stephen @morrySnow @yiguolei