Skip to content

Conversation

@xinyiZzz
Copy link
Contributor

This reverts commit 5fee715.

Proposed changes

#33946 exist two error:

  1. unable to use jemalloc heap profile
  2. maybe cause jemalloc memory not to be freed.

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@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.

@xinyiZzz
Copy link
Contributor Author

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17620	4271	4225	4225
q2	2009	187	191	187
q3	10475	1185	1165	1165
q4	10205	816	744	744
q5	7481	2717	2662	2662
q6	216	130	136	130
q7	1034	595	585	585
q8	9231	2059	2030	2030
q9	8746	6615	6569	6569
q10	8721	3728	3733	3728
q11	448	231	235	231
q12	442	218	227	218
q13	18542	2921	2980	2921
q14	261	233	236	233
q15	535	475	470	470
q16	488	384	379	379
q17	954	735	738	735
q18	8073	7404	7329	7329
q19	6477	1532	1478	1478
q20	627	311	309	309
q21	4972	3336	3232	3232
q22	344	268	270	268
Total cold run time: 117901 ms
Total hot run time: 39828 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4367	4236	4207	4207
q2	387	252	275	252
q3	3002	2794	2694	2694
q4	1900	1567	1601	1567
q5	5371	5368	5291	5291
q6	210	124	125	124
q7	2221	1845	1891	1845
q8	3226	3355	3377	3355
q9	8593	8580	8606	8580
q10	3888	3755	3681	3681
q11	573	482	477	477
q12	776	590	563	563
q13	17575	2953	2976	2953
q14	294	266	276	266
q15	527	464	460	460
q16	488	410	423	410
q17	1776	1490	1452	1452
q18	7722	7446	7272	7272
q19	1635	1545	1544	1544
q20	1954	1761	1794	1761
q21	6990	4754	4808	4754
q22	568	455	486	455
Total cold run time: 74043 ms
Total hot run time: 53963 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 188158 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 691421c630cfd5f4a85852695389cd3204a75af9, data reload: false

query1	909	349	344	344
query2	7849	2343	2377	2343
query3	6649	206	202	202
query4	23478	21680	21427	21427
query5	4182	415	408	408
query6	270	169	174	169
query7	4589	292	294	292
query8	241	186	188	186
query9	8615	2372	2351	2351
query10	593	257	261	257
query11	14736	14160	14197	14160
query12	138	96	95	95
query13	1649	373	364	364
query14	10552	8392	8477	8392
query15	251	184	194	184
query16	8167	268	259	259
query17	1954	555	542	542
query18	2104	269	268	268
query19	352	146	150	146
query20	93	83	84	83
query21	193	127	123	123
query22	5104	4871	4806	4806
query23	33874	33163	32783	32783
query24	11763	2954	2887	2887
query25	627	366	392	366
query26	1725	164	146	146
query27	2970	314	310	310
query28	7575	1981	1989	1981
query29	993	583	586	583
query30	283	153	148	148
query31	957	746	742	742
query32	96	55	52	52
query33	751	250	236	236
query34	1098	491	473	473
query35	833	686	678	678
query36	1067	933	890	890
query37	274	68	66	66
query38	3328	3172	3216	3172
query39	1576	1539	1534	1534
query40	286	127	120	120
query41	42	39	39	39
query42	104	94	96	94
query43	570	524	528	524
query44	1236	711	721	711
query45	282	258	263	258
query46	1084	747	702	702
query47	1937	1856	1890	1856
query48	406	292	293	292
query49	1170	391	385	385
query50	757	378	383	378
query51	6901	6717	6759	6717
query52	101	91	89	89
query53	352	276	274	274
query54	333	231	227	227
query55	81	69	71	69
query56	237	228	215	215
query57	1241	1130	1138	1130
query58	227	192	198	192
query59	3481	3036	3009	3009
query60	249	231	223	223
query61	91	130	89	89
query62	651	440	423	423
query63	305	280	277	277
query64	9484	7150	7112	7112
query65	3095	3078	3053	3053
query66	1373	365	345	345
query67	15356	15023	14956	14956
query68	5149	529	535	529
query69	469	299	299	299
query70	1189	1138	1135	1135
query71	1371	1265	1265	1265
query72	7223	2661	2436	2436
query73	715	323	324	323
query74	6852	6406	6375	6375
query75	3384	2678	2652	2652
query76	3367	913	960	913
query77	402	255	258	255
query78	10880	10228	10161	10161
query79	3347	517	507	507
query80	2097	434	424	424
query81	509	220	223	220
query82	1405	100	95	95
query83	344	169	170	169
query84	259	88	86	86
query85	1597	339	257	257
query86	468	280	299	280
query87	3460	3240	3277	3240
query88	4801	2410	2406	2406
query89	467	363	373	363
query90	2016	187	187	187
query91	126	96	100	96
query92	65	48	48	48
query93	4317	502	499	499
query94	1239	181	182	181
query95	395	303	301	301
query96	607	270	263	263
query97	3194	2959	2905	2905
query98	243	222	231	222
query99	1264	874	859	859
Total cold run time: 297111 ms
Total hot run time: 188158 ms

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 35.20% (8918/25337)
Line Coverage: 26.97% (73330/271852)
Region Coverage: 26.16% (37893/144878)
Branch Coverage: 22.98% (19300/83976)
Coverage Report: http://coverage.selectdb-in.cc/coverage/691421c630cfd5f4a85852695389cd3204a75af9_691421c630cfd5f4a85852695389cd3204a75af9/report/index.html

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

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

@yiguolei yiguolei merged commit 132d974 into apache:master Apr 25, 2024
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. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants