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

[fix](regression)Add log for Reach limit of connections #45887

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

Jibing-Li
Copy link
Contributor

@Jibing-Li Jibing-Li commented Dec 24, 2024

What problem does this PR solve?

Add log to help investigate Reach limit of connections error.

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

@Thearas
Copy link
Contributor

Thearas commented Dec 24, 2024

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 December 24, 2024 16:05
@Jibing-Li
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17631	6127	6062	6062
q2	2050	288	161	161
q3	10455	1207	757	757
q4	10218	860	425	425
q5	7820	2215	1999	1999
q6	207	182	147	147
q7	904	740	592	592
q8	9239	1379	1199	1199
q9	5179	4824	4986	4824
q10	6737	2307	1864	1864
q11	470	281	246	246
q12	346	366	223	223
q13	17751	3589	2933	2933
q14	240	230	212	212
q15	569	499	505	499
q16	641	633	599	599
q17	609	862	328	328
q18	7097	6611	6470	6470
q19	2010	978	588	588
q20	307	319	191	191
q21	2820	2203	2000	2000
q22	366	337	301	301
Total cold run time: 103666 ms
Total hot run time: 32620 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6293	6211	6243	6211
q2	238	331	225	225
q3	2242	2594	2284	2284
q4	1414	1811	1360	1360
q5	4339	4695	4833	4695
q6	189	177	144	144
q7	2074	1928	1757	1757
q8	2610	2805	2708	2708
q9	7279	7210	7256	7210
q10	3086	3368	2851	2851
q11	600	488	487	487
q12	630	723	591	591
q13	3441	3755	3076	3076
q14	287	309	268	268
q15	561	505	514	505
q16	652	691	634	634
q17	1227	1724	1270	1270
q18	7720	7413	7550	7413
q19	869	1045	1120	1045
q20	1993	2047	1941	1941
q21	5679	5261	4894	4894
q22	598	617	610	610
Total cold run time: 54021 ms
Total hot run time: 52179 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 197114 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 3267484952f11a9623ba5d79923dcc6b4963aad7, data reload: false

query1	1269	932	939	932
query2	6489	2281	2317	2281
query3	11049	4905	4898	4898
query4	32893	23810	23556	23556
query5	3911	612	482	482
query6	260	219	180	180
query7	3996	510	304	304
query8	292	243	227	227
query9	9232	2746	2745	2745
query10	464	315	261	261
query11	18109	15992	15198	15198
query12	161	111	105	105
query13	1588	566	416	416
query14	11392	6870	7118	6870
query15	244	215	193	193
query16	7970	603	481	481
query17	1585	784	604	604
query18	2094	436	347	347
query19	202	197	168	168
query20	141	121	114	114
query21	213	180	106	106
query22	4746	4743	4515	4515
query23	34869	33772	33861	33772
query24	6527	2369	2350	2350
query25	484	475	385	385
query26	795	271	161	161
query27	2093	495	338	338
query28	5665	2536	2448	2448
query29	615	570	428	428
query30	217	188	157	157
query31	990	947	858	858
query32	71	85	58	58
query33	476	384	295	295
query34	804	869	540	540
query35	809	836	754	754
query36	1012	1069	998	998
query37	119	113	80	80
query38	4197	4199	4425	4199
query39	1543	1459	1440	1440
query40	218	129	102	102
query41	42	44	44	44
query42	116	109	103	103
query43	528	557	511	511
query44	1372	804	829	804
query45	189	178	175	175
query46	931	1081	688	688
query47	1995	1978	1891	1891
query48	401	443	337	337
query49	706	483	402	402
query50	672	679	402	402
query51	7249	7416	7244	7244
query52	103	107	102	102
query53	244	272	198	198
query54	504	489	423	423
query55	91	84	88	84
query56	269	278	249	249
query57	1254	1239	1173	1173
query58	242	242	222	222
query59	3073	3288	3009	3009
query60	294	274	258	258
query61	103	105	102	102
query62	875	827	770	770
query63	239	200	200	200
query64	3386	1035	673	673
query65	3376	3291	3283	3283
query66	983	412	301	301
query67	16511	15995	15550	15550
query68	9801	827	517	517
query69	475	298	257	257
query70	1197	1156	1125	1125
query71	435	293	258	258
query72	5972	3830	3900	3830
query73	678	767	360	360
query74	9813	8932	8840	8840
query75	4656	3157	2667	2667
query76	5193	1198	856	856
query77	932	382	266	266
query78	10292	10193	9565	9565
query79	4369	887	576	576
query80	743	519	424	424
query81	491	284	226	226
query82	613	157	126	126
query83	194	158	142	142
query84	285	98	75	75
query85	776	354	293	293
query86	360	323	294	294
query87	4564	4341	4529	4341
query88	3770	2236	2224	2224
query89	456	326	295	295
query90	1972	185	187	185
query91	135	134	105	105
query92	70	53	51	51
query93	2564	885	533	533
query94	667	411	293	293
query95	337	263	244	244
query96	498	605	276	276
query97	2766	2795	2709	2709
query98	227	216	201	201
query99	1726	1548	1416	1416
Total cold run time: 302579 ms
Total hot run time: 197114 ms

@doris-robot
Copy link

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

query1	0.03	0.03	0.02
query2	0.07	0.04	0.03
query3	0.24	0.07	0.07
query4	1.61	0.10	0.10
query5	0.43	0.41	0.41
query6	1.17	0.68	0.68
query7	0.02	0.01	0.01
query8	0.04	0.03	0.04
query9	0.60	0.50	0.52
query10	0.55	0.56	0.56
query11	0.15	0.10	0.11
query12	0.14	0.11	0.12
query13	0.63	0.59	0.62
query14	2.72	2.77	2.75
query15	0.91	0.85	0.85
query16	0.41	0.38	0.40
query17	0.99	1.08	1.05
query18	0.22	0.21	0.20
query19	1.90	1.83	1.96
query20	0.01	0.01	0.01
query21	15.38	0.93	0.57
query22	0.75	0.76	0.70
query23	15.30	1.46	0.53
query24	2.68	1.38	1.32
query25	0.17	0.20	0.07
query26	0.25	0.14	0.13
query27	0.06	0.06	0.05
query28	14.47	1.65	1.06
query29	12.61	4.00	3.26
query30	0.25	0.10	0.07
query31	2.82	0.63	0.40
query32	3.23	0.58	0.48
query33	3.11	3.17	3.15
query34	16.74	5.35	4.58
query35	4.57	4.57	4.59
query36	0.67	0.51	0.49
query37	0.10	0.06	0.06
query38	0.05	0.04	0.04
query39	0.03	0.03	0.02
query40	0.16	0.13	0.14
query41	0.08	0.03	0.03
query42	0.04	0.03	0.02
query43	0.04	0.03	0.03
Total cold run time: 106.4 s
Total hot run time: 32.04 s

Copy link
Contributor

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

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

PR approved by anyone and no changes requested.

Copy link
Contributor

@zfr9527 zfr9527 left a comment

Choose a reason for hiding this comment

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

LGTM

@Jibing-Li Jibing-Li merged commit 5d0e510 into apache:master Dec 25, 2024
28 of 29 checks passed
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.

5 participants