Skip to content
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.

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

[chore](compile check) Fix compile check of exec_env.cpp #46847

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zhiqiang-hhhh
Copy link
Contributor

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

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

@hello-stephen
Copy link
Contributor

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?

@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17570	6142	6074	6074
q2	2057	299	174	174
q3	10425	1286	743	743
q4	10229	858	435	435
q5	8073	2189	1978	1978
q6	206	181	152	152
q7	906	756	603	603
q8	9249	1344	1139	1139
q9	5125	4863	4866	4863
q10	6830	2288	1874	1874
q11	491	295	265	265
q12	360	360	241	241
q13	17921	3750	3238	3238
q14	237	245	220	220
q15	566	512	492	492
q16	638	627	603	603
q17	571	849	340	340
q18	6984	6510	6394	6394
q19	1656	950	563	563
q20	316	330	191	191
q21	2832	2292	1997	1997
q22	365	333	312	312
Total cold run time: 103607 ms
Total hot run time: 32891 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6322	6270	6261	6261
q2	240	329	237	237
q3	2300	2693	2368	2368
q4	1380	1832	1338	1338
q5	4336	4744	4765	4744
q6	190	174	143	143
q7	2078	2023	1827	1827
q8	2900	2878	2706	2706
q9	7307	7267	7270	7267
q10	3074	3290	2753	2753
q11	599	543	499	499
q12	724	763	659	659
q13	3420	3836	3335	3335
q14	291	320	263	263
q15	585	515	523	515
q16	650	720	648	648
q17	1258	1743	1278	1278
q18	7742	7530	7365	7365
q19	825	1149	1098	1098
q20	1969	2019	1879	1879
q21	5658	5134	5005	5005
q22	630	645	598	598
Total cold run time: 54478 ms
Total hot run time: 52786 ms

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 39.35% (10253/26053)
Line Coverage: 30.52% (87419/286408)
Region Coverage: 29.55% (44531/150713)
Branch Coverage: 26.10% (22795/87324)
Coverage Report: http://coverage.selectdb-in.cc/coverage/e417724a0021b975b271696a8b07ebdbcfa30dbb_e417724a0021b975b271696a8b07ebdbcfa30dbb/report/index.html

@doris-robot
Copy link

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

query1	1327	991	926	926
query2	6180	2287	2287	2287
query3	10980	4769	4436	4436
query4	33154	23462	23606	23462
query5	4902	604	437	437
query6	283	198	201	198
query7	3999	486	306	306
query8	323	246	230	230
query9	9472	2723	2729	2723
query10	487	301	249	249
query11	18104	15601	15097	15097
query12	156	108	105	105
query13	1628	535	398	398
query14	10960	7047	6896	6896
query15	241	243	189	189
query16	8082	636	455	455
query17	1507	737	572	572
query18	2155	415	317	317
query19	208	195	198	195
query20	119	115	115	115
query21	213	120	109	109
query22	4448	4521	4400	4400
query23	34249	33068	33444	33068
query24	6370	2338	2276	2276
query25	519	478	388	388
query26	814	291	154	154
query27	2122	459	336	336
query28	6010	2491	2510	2491
query29	634	558	457	457
query30	226	184	161	161
query31	958	852	819	819
query32	80	59	55	55
query33	485	381	301	301
query34	747	884	527	527
query35	802	821	722	722
query36	1020	1019	933	933
query37	118	103	75	75
query38	4112	4285	4094	4094
query39	1508	1466	1445	1445
query40	208	117	103	103
query41	59	56	54	54
query42	130	102	103	102
query43	514	530	492	492
query44	1335	857	856	856
query45	189	195	181	181
query46	871	1079	655	655
query47	1891	1914	1885	1885
query48	387	396	317	317
query49	715	493	416	416
query50	623	693	398	398
query51	7092	7144	6922	6922
query52	104	100	93	93
query53	233	261	193	193
query54	495	518	443	443
query55	86	82	81	81
query56	258	280	250	250
query57	1198	1165	1114	1114
query58	239	248	240	240
query59	3143	3195	3098	3098
query60	294	262	254	254
query61	118	118	120	118
query62	823	810	750	750
query63	236	202	191	191
query64	3409	1040	663	663
query65	3249	3245	3250	3245
query66	811	422	315	315
query67	16345	15755	15388	15388
query68	8474	695	515	515
query69	486	288	251	251
query70	1205	1143	1127	1127
query71	453	283	263	263
query72	6463	3809	3850	3809
query73	664	761	365	365
query74	10592	9380	8976	8976
query75	4101	3116	2645	2645
query76	3818	1156	753	753
query77	791	372	274	274
query78	10119	10026	9361	9361
query79	3675	807	594	594
query80	621	519	445	445
query81	497	267	238	238
query82	677	149	113	113
query83	166	176	154	154
query84	252	99	73	73
query85	777	348	296	296
query86	366	273	298	273
query87	4392	4347	4289	4289
query88	4783	2204	2190	2190
query89	410	322	284	284
query90	1827	185	188	185
query91	138	150	108	108
query92	71	53	52	52
query93	2137	877	519	519
query94	651	395	291	291
query95	334	263	252	252
query96	498	605	276	276
query97	2870	2906	2768	2768
query98	226	197	200	197
query99	1441	1524	1398	1398
Total cold run time: 298732 ms
Total hot run time: 194555 ms

@doris-robot
Copy link

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

query1	0.04	0.03	0.04
query2	0.07	0.04	0.04
query3	0.24	0.07	0.07
query4	1.62	0.10	0.10
query5	0.42	0.43	0.41
query6	1.16	0.65	0.65
query7	0.02	0.02	0.01
query8	0.04	0.03	0.04
query9	0.58	0.51	0.50
query10	0.55	0.55	0.56
query11	0.14	0.10	0.10
query12	0.15	0.11	0.11
query13	0.61	0.60	0.60
query14	2.73	2.71	2.75
query15	0.90	0.82	0.83
query16	0.38	0.39	0.39
query17	1.06	1.04	1.06
query18	0.23	0.21	0.21
query19	1.95	1.86	2.09
query20	0.02	0.01	0.02
query21	15.35	0.92	0.59
query22	0.75	0.74	0.64
query23	15.39	1.38	0.50
query24	2.85	0.87	1.14
query25	0.18	0.05	0.17
query26	0.28	0.16	0.14
query27	0.06	0.04	0.04
query28	13.61	1.57	1.05
query29	12.55	4.00	3.27
query30	0.26	0.09	0.06
query31	2.82	0.62	0.39
query32	3.23	0.54	0.46
query33	3.10	3.05	3.15
query34	16.56	5.08	4.50
query35	4.49	4.50	4.48
query36	0.64	0.49	0.48
query37	0.08	0.06	0.06
query38	0.04	0.04	0.04
query39	0.03	0.03	0.02
query40	0.16	0.13	0.12
query41	0.08	0.02	0.02
query42	0.03	0.02	0.02
query43	0.03	0.03	0.03
Total cold run time: 105.48 s
Total hot run time: 31.12 s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants