Skip to content

[Exec](alias) support stddev alias to std #52819

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
Jul 6, 2025
Merged

[Exec](alias) support stddev alias to std #52819

merged 1 commit into from
Jul 6, 2025

Conversation

@HappenLee
Copy link
Contributor

@HappenLee HappenLee commented Jul 5, 2025 •
edited
Loading

What problem does this PR solve?

Support stddev alias to std for BI support like mysql way

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?

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 Jul 5, 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?

@HappenLee
Copy link
Contributor Author

HappenLee commented Jul 5, 2025

run buildall

@HappenLee HappenLee marked this pull request as ready for review July 5, 2025 05:25
@doris-robot
Copy link

doris-robot commented Jul 5, 2025

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

------ Round 1 ----------------------------------
q1	17575	5143	5050	5050
q2	1931	285	185	185
q3	10353	1314	692	692
q4	10298	1034	498	498
q5	8588	2404	2305	2305
q6	187	161	125	125
q7	920	728	598	598
q8	9297	1298	1043	1043
q9	6803	5150	5268	5150
q10	6884	2401	1947	1947
q11	481	292	264	264
q12	345	367	213	213
q13	17759	3654	3076	3076
q14	218	216	211	211
q15	543	486	479	479
q16	419	426	382	382
q17	598	871	351	351
q18	7486	7172	7174	7172
q19	1357	979	557	557
q20	313	338	213	213
q21	3870	3154	2311	2311
q22	1053	1029	960	960
Total cold run time: 107278 ms
Total hot run time: 33782 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5164	5131	5100	5100
q2	234	324	215	215
q3	2175	2621	2299	2299
q4	1382	1741	1346	1346
q5	4229	4582	4451	4451
q6	207	168	128	128
q7	2037	1952	1793	1793
q8	2667	2494	2620	2494
q9	7433	7358	7462	7358
q10	3225	3352	2904	2904
q11	611	517	493	493
q12	707	796	648	648
q13	3726	4147	3476	3476
q14	304	313	318	313
q15	579	497	475	475
q16	453	543	486	486
q17	1183	1627	1429	1429
q18	8057	7914	7146	7146
q19	771	785	795	785
q20	1917	1974	1836	1836
q21	4694	4299	4353	4299
q22	1078	1023	1006	1006
Total cold run time: 52833 ms
Total hot run time: 50480 ms

@doris-robot
Copy link

doris-robot commented Jul 5, 2025

TPC-DS: Total hot run time: 184369 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 ccf95b6eb74a2513967812a1f05461ff496d87e2, data reload: false

query1	1001	388	371	371
query2	6533	1689	1705	1689
query3	6740	220	225	220
query4	26203	24001	23281	23281
query5	4474	603	440	440
query6	303	212	203	203
query7	4629	483	279	279
query8	281	209	200	200
query9	8651	2581	2580	2580
query10	460	333	281	281
query11	15528	15401	14797	14797
query12	159	108	106	106
query13	1653	528	395	395
query14	8777	5533	5602	5533
query15	198	187	180	180
query16	7175	630	473	473
query17	1181	714	587	587
query18	1972	399	290	290
query19	189	197	164	164
query20	124	121	109	109
query21	215	121	105	105
query22	4180	4297	4041	4041
query23	33795	33041	32924	32924
query24	8411	2392	2363	2363
query25	530	472	383	383
query26	1227	273	147	147
query27	2756	512	336	336
query28	4344	2122	2074	2074
query29	778	548	429	429
query30	282	221	188	188
query31	898	838	738	738
query32	69	64	57	57
query33	548	351	310	310
query34	789	839	531	531
query35	772	800	757	757
query36	913	1001	887	887
query37	110	98	69	69
query38	4176	4118	4078	4078
query39	1453	1416	1430	1416
query40	208	113	100	100
query41	55	54	57	54
query42	127	112	104	104
query43	475	486	459	459
query44	1294	814	809	809
query45	183	169	164	164
query46	843	992	626	626
query47	1798	1825	1724	1724
query48	376	399	317	317
query49	766	474	389	389
query50	631	689	412	412
query51	4136	4134	4125	4125
query52	109	107	95	95
query53	217	249	190	190
query54	606	570	511	511
query55	83	80	83	80
query56	317	301	290	290
query57	1193	1218	1169	1169
query58	279	263	260	260
query59	2625	2591	2591	2591
query60	354	314	309	309
query61	148	142	143	142
query62	817	719	644	644
query63	225	188	182	182
query64	4316	948	633	633
query65	4262	4146	4203	4146
query66	1141	439	317	317
query67	15684	15542	15287	15287
query68	7832	867	522	522
query69	467	300	264	264
query70	1235	1126	1074	1074
query71	446	333	306	306
query72	5602	4754	4944	4754
query73	676	663	343	343
query74	9078	9214	8929	8929
query75	3584	3184	2687	2687
query76	3380	1140	706	706
query77	781	381	285	285
query78	9941	10150	9261	9261
query79	2020	813	579	579
query80	586	508	429	429
query81	454	262	283	262
query82	417	130	97	97
query83	252	250	231	231
query84	245	102	92	92
query85	776	354	310	310
query86	327	276	293	276
query87	4353	4517	4326	4326
query88	3441	2250	2268	2250
query89	372	318	280	280
query90	1926	205	200	200
query91	131	143	110	110
query92	71	61	56	56
query93	1140	920	581	581
query94	687	408	317	317
query95	367	282	286	282
query96	484	571	275	275
query97	2741	2803	2673	2673
query98	225	203	200	200
query99	1478	1376	1272	1272
Total cold run time: 271260 ms
Total hot run time: 184369 ms

@doris-robot
Copy link

doris-robot commented Jul 5, 2025

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

query1	0.05	0.04	0.04
query2	0.09	0.04	0.05
query3	0.24	0.07	0.08
query4	1.61	0.12	0.11
query5	0.45	0.44	0.40
query6	1.14	0.66	0.66
query7	0.02	0.01	0.02
query8	0.04	0.03	0.04
query9	0.60	0.52	0.52
query10	0.57	0.59	0.58
query11	0.16	0.11	0.12
query12	0.15	0.12	0.12
query13	0.63	0.61	0.60
query14	0.79	0.81	0.82
query15	0.89	0.89	0.86
query16	0.38	0.39	0.38
query17	1.06	1.06	1.09
query18	0.23	0.22	0.21
query19	1.93	1.85	1.84
query20	0.02	0.01	0.02
query21	15.39	0.86	0.54
query22	0.77	1.19	0.61
query23	15.01	1.40	0.65
query24	7.58	1.03	1.15
query25	0.50	0.18	0.21
query26	0.72	0.16	0.15
query27	0.05	0.06	0.05
query28	9.55	0.85	0.44
query29	12.55	3.96	3.30
query30	0.25	0.09	0.06
query31	2.85	0.61	0.38
query32	3.24	0.55	0.46
query33	3.04	3.12	3.07
query34	16.14	5.38	4.79
query35	4.84	4.84	4.81
query36	0.66	0.51	0.49
query37	0.09	0.07	0.07
query38	0.05	0.04	0.04
query39	0.04	0.03	0.02
query40	0.18	0.15	0.14
query41	0.09	0.02	0.03
query42	0.03	0.02	0.02
query43	0.04	0.03	0.03
Total cold run time: 104.71 s
Total hot run time: 29.81 s

@hello-stephen
Copy link
Contributor

hello-stephen commented Jul 5, 2025

BE UT Coverage Report

Increment line coverage 100.00% (1/1) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 57.34% (15484/27006)
Line Coverage 46.31% (140670/303772)
Region Coverage 45.60% (71181/156112)
Branch Coverage 40.31% (37503/93036)

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

github-actions bot commented Jul 5, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 5, 2025

PR approved by anyone and no changes requested.

@BiteTheDDDDt BiteTheDDDDt merged commit a1b7c42 into apache:master Jul 6, 2025
30 of 34 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 7, 2025
@HappenLee
[Exec](alias) support stddev alias to std (#52819)
e7ebdc0 
Support `stddev` alias to `std` for BI support like mysql way
@github-actions github-actions bot mentioned this pull request Jul 7, 2025
Merged
github-actions bot pushed a commit that referenced this pull request Jul 7, 2025
@HappenLee
[Exec](alias) support stddev alias to std (#52819)
5e75164 
Support `stddev` alias to `std` for BI support like mysql way
@github-actions github-actions bot mentioned this pull request Jul 7, 2025
Merged
dataroaring pushed a commit that referenced this pull request Jul 7, 2025
@github-actions @HappenLee
branch-3.0: [Exec](alias) support stddev alias to std #52819 (#52838)
e78970f 
Cherry-picked from #52819

Co-authored-by: HappenLee <[email protected]>
morrySnow pushed a commit that referenced this pull request Jul 8, 2025
@github-actions @HappenLee
branch-3.1: [Exec](alias) support stddev alias to std #52819 (#52866)
f639783 
Cherry-picked from #52819

Co-authored-by: HappenLee <[email protected]>
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

@yiguolei yiguolei yiguolei approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@HappenLee @Thearas @doris-robot @hello-stephen @BiteTheDDDDt @yiguolei @dataroaring @morrySnow