Skip to content

Conversation

@HappenLee
Copy link
Contributor

Proposed changes

  1. delete the file:be/src/vec/utils/count_by_enum_helpers.hpp
  2. move code to:be/src/vec/aggregate_functions/aggregate_function_count_by_enum.h

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@HappenLee
Copy link
Contributor Author

run buildall

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions


#pragma once

#include <rapidjson/document.h>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: 'rapidjson/document.h' file not found [clang-diagnostic-error]

#include <rapidjson/document.h>
         ^

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	18016	4532	4415	4415
q2	2222	190	201	190
q3	10439	1250	1075	1075
q4	10188	813	812	812
q5	7611	2727	2644	2644
q6	217	139	139	139
q7	956	602	604	602
q8	9227	2104	2112	2104
q9	8708	6629	6611	6611
q10	8678	3755	3779	3755
q11	463	236	243	236
q12	395	230	217	217
q13	18972	2978	2984	2978
q14	280	249	240	240
q15	519	489	498	489
q16	475	404	387	387
q17	993	666	719	666
q18	8344	7502	7426	7426
q19	7665	1460	1442	1442
q20	691	308	321	308
q21	4933	3197	3237	3197
q22	364	299	292	292
Total cold run time: 120356 ms
Total hot run time: 40225 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4410	4276	4253	4253
q2	376	261	270	261
q3	3034	2747	2762	2747
q4	1851	1586	1558	1558
q5	5300	5342	5333	5333
q6	220	134	134	134
q7	2130	1756	1755	1755
q8	3217	3676	3345	3345
q9	8455	8418	8390	8390
q10	3956	3697	3702	3697
q11	606	483	507	483
q12	777	631	618	618
q13	17542	3005	2981	2981
q14	317	275	273	273
q15	521	479	475	475
q16	487	414	410	410
q17	1805	1472	1459	1459
q18	7789	7558	7388	7388
q19	3145	1673	1607	1607
q20	2032	1819	1772	1772
q21	5055	4572	4768	4572
q22	621	513	504	504
Total cold run time: 73646 ms
Total hot run time: 54015 ms

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17636	4435	4347	4347
q2	2017	190	188	188
q3	10453	1185	1049	1049
q4	10176	851	822	822
q5	7545	2718	2699	2699
q6	224	134	135	134
q7	958	591	591	591
q8	9209	2073	2101	2073
q9	8689	6615	6606	6606
q10	8730	3778	3782	3778
q11	442	231	233	231
q12	405	218	222	218
q13	17922	2969	2954	2954
q14	282	229	240	229
q15	517	486	472	472
q16	500	392	370	370
q17	985	694	657	657
q18	8112	7500	7429	7429
q19	6959	1407	1353	1353
q20	669	309	320	309
q21	4852	3053	3178	3053
q22	344	283	281	281
Total cold run time: 117626 ms
Total hot run time: 39843 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4501	4303	4282	4282
q2	381	270	276	270
q3	3019	2852	2942	2852
q4	1940	1663	1714	1663
q5	5323	5469	5455	5455
q6	235	132	134	132
q7	2225	1819	1711	1711
q8	3191	3308	3339	3308
q9	8417	8378	8422	8378
q10	3937	3745	3733	3733
q11	584	509	486	486
q12	766	614	611	611
q13	16407	3227	2978	2978
q14	302	279	271	271
q15	506	478	472	472
q16	449	399	404	399
q17	1784	1491	1497	1491
q18	7716	7463	7348	7348
q19	1738	1507	1639	1507
q20	2117	1787	1768	1768
q21	4867	4657	4622	4622
q22	608	495	496	495
Total cold run time: 71013 ms
Total hot run time: 54232 ms

@HappenLee
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17608	4343	4320	4320
q2	2011	194	189	189
q3	10438	1232	1149	1149
q4	10193	894	795	795
q5	7579	2697	2637	2637
q6	222	139	137	137
q7	955	598	598	598
q8	9220	2068	2084	2068
q9	8700	6638	6585	6585
q10	8887	3746	3821	3746
q11	437	233	233	233
q12	437	232	216	216
q13	18802	2947	2981	2947
q14	297	234	234	234
q15	515	477	474	474
q16	500	370	376	370
q17	973	604	657	604
q18	8215	7470	7395	7395
q19	5015	1435	1312	1312
q20	707	318	329	318
q21	4855	3187	3261	3187
q22	340	279	283	279
Total cold run time: 116906 ms
Total hot run time: 39793 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4400	4225	4226	4225
q2	379	271	265	265
q3	2998	2772	2949	2772
q4	1925	1643	1729	1643
q5	5397	5477	5405	5405
q6	218	133	132	132
q7	2241	1856	1717	1717
q8	3145	3313	3617	3313
q9	8430	8561	8431	8431
q10	3959	3719	3636	3636
q11	576	492	488	488
q12	772	582	598	582
q13	16289	3189	2976	2976
q14	301	283	268	268
q15	527	474	481	474
q16	473	430	431	430
q17	1754	1492	1498	1492
q18	7741	7410	7353	7353
q19	1703	1494	1446	1446
q20	2066	1771	1755	1755
q21	4861	4687	4614	4614
q22	596	517	498	498
Total cold run time: 70751 ms
Total hot run time: 53915 ms

@HappenLee
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17646	4387	4255	4255
q2	2013	187	193	187
q3	10440	1217	1129	1129
q4	10183	802	650	650
q5	7554	2678	2668	2668
q6	223	140	135	135
q7	943	612	592	592
q8	9220	2048	2029	2029
q9	8669	6569	6533	6533
q10	8911	3796	3781	3781
q11	445	241	231	231
q12	465	226	222	222
q13	18713	2994	2995	2994
q14	274	236	238	236
q15	504	489	482	482
q16	503	391	377	377
q17	962	626	749	626
q18	7994	7450	7417	7417
q19	8242	1438	1329	1329
q20	677	319	316	316
q21	5061	3079	3236	3079
q22	331	275	292	275
Total cold run time: 119973 ms
Total hot run time: 39543 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4349	4243	4217	4217
q2	374	262	274	262
q3	2988	2940	2841	2841
q4	1908	1680	1632	1632
q5	5338	5401	5486	5401
q6	226	127	130	127
q7	2265	1755	1669	1669
q8	3107	3297	3304	3297
q9	8456	8389	8356	8356
q10	3903	3732	3698	3698
q11	579	480	483	480
q12	784	603	586	586
q13	17920	3227	3193	3193
q14	319	288	281	281
q15	511	478	468	468
q16	490	414	421	414
q17	1817	1474	1473	1473
q18	7770	7413	7255	7255
q19	4158	1633	1567	1567
q20	2117	1785	1758	1758
q21	4891	4651	4842	4651
q22	592	519	537	519
Total cold run time: 74862 ms
Total hot run time: 54145 ms

@HappenLee
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17964	4457	4412	4412
q2	2976	196	207	196
q3	10873	1215	1153	1153
q4	10472	882	824	824
q5	7745	2739	2700	2700
q6	221	141	138	138
q7	979	620	614	614
q8	9470	2102	2119	2102
q9	8663	6578	6538	6538
q10	8777	3773	3747	3747
q11	451	239	245	239
q12	401	233	229	229
q13	17767	2994	3013	2994
q14	281	238	228	228
q15	527	480	488	480
q16	493	381	387	381
q17	973	670	698	670
q18	8035	7498	7389	7389
q19	4939	1460	1448	1448
q20	693	326	329	326
q21	4961	3909	3377	3377
q22	358	289	288	288
Total cold run time: 118019 ms
Total hot run time: 40473 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4371	4224	4226	4224
q2	364	260	257	257
q3	3028	2775	2750	2750
q4	1909	1568	1641	1568
q5	5316	5357	5305	5305
q6	217	132	131	131
q7	2105	1731	1718	1718
q8	3172	3352	3320	3320
q9	8477	8423	8400	8400
q10	3941	3775	3692	3692
q11	590	484	521	484
q12	785	599	585	585
q13	16470	3020	3021	3020
q14	309	279	280	279
q15	520	484	477	477
q16	471	421	415	415
q17	1768	1485	1478	1478
q18	7625	7423	7496	7423
q19	1645	1696	1551	1551
q20	2009	1810	1778	1778
q21	4941	4661	4891	4661
q22	571	525	498	498
Total cold run time: 70604 ms
Total hot run time: 54014 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 173092 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 78a616b7d285017e085a8d9628dd58f2b8fec142, data reload: false

query1	907	367	355	355
query2	6444	1861	1812	1812
query3	6649	207	220	207
query4	24662	17682	17383	17383
query5	4210	478	487	478
query6	273	178	163	163
query7	4592	292	283	283
query8	245	196	192	192
query9	8559	2430	2402	2402
query10	449	284	275	275
query11	11736	10205	10119	10119
query12	139	92	90	90
query13	1645	380	372	372
query14	10185	7446	7050	7050
query15	223	166	164	164
query16	7814	463	516	463
query17	1379	559	556	556
query18	1978	293	298	293
query19	198	156	156	156
query20	92	85	82	82
query21	209	126	123	123
query22	4373	4210	4068	4068
query23	33982	33240	33244	33240
query24	12102	2897	2868	2868
query25	693	389	393	389
query26	1775	158	153	153
query27	2929	276	284	276
query28	7441	2035	2011	2011
query29	1117	652	626	626
query30	296	149	147	147
query31	991	750	757	750
query32	95	57	59	57
query33	785	348	347	347
query34	915	481	492	481
query35	871	766	753	753
query36	1109	944	952	944
query37	177	82	83	82
query38	2853	2746	2774	2746
query39	873	835	809	809
query40	286	124	120	120
query41	53	48	46	46
query42	124	102	104	102
query43	493	459	465	459
query44	1208	730	728	728
query45	194	166	163	163
query46	1093	747	724	724
query47	1849	1753	1805	1753
query48	370	299	310	299
query49	1188	421	429	421
query50	801	411	491	411
query51	6928	6647	6529	6529
query52	108	89	94	89
query53	371	286	286	286
query54	979	453	469	453
query55	75	73	73	73
query56	281	282	270	270
query57	1133	1074	1087	1074
query58	268	243	257	243
query59	2754	2853	2714	2714
query60	305	280	282	280
query61	97	97	93	93
query62	831	661	645	645
query63	326	297	301	297
query64	10466	2189	1657	1657
query65	3160	3110	3097	3097
query66	1385	323	332	323
query67	15887	15072	14973	14973
query68	6375	558	551	551
query69	722	443	368	368
query70	1238	1062	1089	1062
query71	524	278	274	274
query72	8559	5591	5601	5591
query73	816	322	325	322
query74	6161	5788	5675	5675
query75	5367	2687	2722	2687
query76	4662	931	899	899
query77	759	312	313	312
query78	9743	9029	8961	8961
query79	8149	525	531	525
query80	2285	472	484	472
query81	599	227	231	227
query82	1063	134	132	132
query83	299	164	163	163
query84	274	86	84	84
query85	1435	306	316	306
query86	439	307	313	307
query87	3380	3137	3132	3132
query88	4814	2365	2366	2365
query89	496	391	377	377
query90	2040	195	194	194
query91	133	102	100	100
query92	64	49	52	49
query93	6341	526	517	517
query94	1348	300	294	294
query95	408	319	311	311
query96	637	272	274	272
query97	3221	3000	3041	3000
query98	220	197	195	195
query99	1557	1308	1281	1281
Total cold run time: 303106 ms
Total hot run time: 173092 ms

@doris-robot
Copy link

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

query1	0.04	0.04	0.03
query2	0.08	0.04	0.04
query3	0.22	0.05	0.05
query4	1.68	0.08	0.07
query5	0.48	0.48	0.49
query6	1.12	0.73	0.72
query7	0.02	0.02	0.01
query8	0.05	0.04	0.04
query9	0.55	0.49	0.50
query10	0.55	0.54	0.55
query11	0.16	0.12	0.12
query12	0.15	0.13	0.12
query13	0.60	0.58	0.58
query14	0.77	0.78	0.79
query15	0.86	0.81	0.82
query16	0.35	0.36	0.36
query17	0.97	0.94	1.04
query18	0.22	0.21	0.22
query19	1.75	1.70	1.68
query20	0.01	0.01	0.01
query21	15.40	0.79	0.65
query22	4.44	7.57	1.90
query23	18.28	1.37	1.29
query24	2.07	0.24	0.21
query25	0.15	0.09	0.09
query26	0.29	0.22	0.21
query27	0.46	0.25	0.23
query28	13.30	1.02	0.99
query29	12.54	3.30	3.30
query30	0.26	0.07	0.05
query31	2.89	0.40	0.40
query32	3.25	0.47	0.47
query33	2.88	2.90	2.92
query34	17.01	4.37	4.37
query35	4.44	4.43	4.47
query36	0.65	0.48	0.48
query37	0.19	0.16	0.16
query38	0.16	0.14	0.15
query39	0.04	0.04	0.03
query40	0.16	0.13	0.13
query41	0.09	0.05	0.04
query42	0.05	0.05	0.04
query43	0.04	0.04	0.04
Total cold run time: 109.67 s
Total hot run time: 30.63 s

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

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@HappenLee HappenLee merged commit 8d0b7f1 into apache:master Jul 24, 2024
dataroaring pushed a commit that referenced this pull request Aug 11, 2024
…8193)

1. delete the file:be/src/vec/utils/count_by_enum_helpers.hpp
2. move code
to:be/src/vec/aggregate_functions/aggregate_function_count_by_enum.h
dataroaring pushed a commit that referenced this pull request Aug 16, 2024
…8193)

1. delete the file:be/src/vec/utils/count_by_enum_helpers.hpp
2. move code
to:be/src/vec/aggregate_functions/aggregate_function_count_by_enum.h
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants