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

branch-3.0: [fix](config) fe config sync_image_timeout_second should not be masterOnly #43954 #44383

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Cherry-picked from #43954

…rOnly (#43954)

### What problem does this PR solve?
sync_image_timeout_second is for the follower,should not be  masterOnly

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

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?

@dataroaring dataroaring reopened this Nov 21, 2024
@doris-robot
Copy link

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17602	7359	7261	7261
q2	2053	161	154	154
q3	10841	1084	1117	1084
q4	10564	706	725	706
q5	7752	2815	2679	2679
q6	235	148	143	143
q7	949	601	599	599
q8	9566	1938	1911	1911
q9	8100	6321	6371	6321
q10	6956	2263	2279	2263
q11	451	261	256	256
q12	402	217	204	204
q13	17777	2953	2962	2953
q14	242	219	214	214
q15	548	513	525	513
q16	679	598	602	598
q17	961	523	517	517
q18	7128	6629	6662	6629
q19	4048	1052	971	971
q20	484	193	187	187
q21	4052	3076	3056	3056
q22	1068	973	948	948
Total cold run time: 112458 ms
Total hot run time: 40167 ms

----- Round 2, with runtime_filter_mode=off -----
q1	7417	7189	7183	7183
q2	316	221	218	218
q3	2996	2903	2866	2866
q4	2097	1748	1762	1748
q5	5623	5669	5769	5669
q6	223	138	140	138
q7	2181	1771	1729	1729
q8	3300	3428	3339	3339
q9	8851	8830	8777	8777
q10	3533	3482	3468	3468
q11	589	503	496	496
q12	833	637	603	603
q13	16440	3119	3154	3119
q14	296	271	287	271
q15	546	524	519	519
q16	710	664	660	660
q17	1827	1606	1577	1577
q18	8171	7764	7498	7498
q19	1684	1478	1661	1478
q20	2055	1856	1788	1788
q21	5350	5238	5160	5160
q22	1116	1070	1043	1043
Total cold run time: 76154 ms
Total hot run time: 59347 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 194312 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 613bcd0ad695660947751efb898fcc4085379d4b, data reload: false

query1	1224	917	891	891
query2	6249	2130	2025	2025
query3	10781	3769	3891	3769
query4	68451	29631	23447	23447
query5	5518	446	432	432
query6	437	182	169	169
query7	5968	312	302	302
query8	322	233	222	222
query9	9436	2628	2628	2628
query10	507	269	258	258
query11	17856	15325	15616	15325
query12	168	107	101	101
query13	1535	431	434	431
query14	11178	6378	7248	6378
query15	217	179	195	179
query16	7935	485	472	472
query17	1074	544	546	544
query18	1893	284	291	284
query19	192	147	155	147
query20	115	113	111	111
query21	214	99	102	99
query22	4734	4851	4635	4635
query23	35189	33726	34064	33726
query24	5948	2863	2856	2856
query25	518	399	400	399
query26	680	165	167	165
query27	1941	303	305	303
query28	4282	2558	2508	2508
query29	712	468	451	451
query30	243	159	166	159
query31	1004	811	843	811
query32	64	56	61	56
query33	423	282	287	282
query34	898	491	509	491
query35	849	740	725	725
query36	1078	956	927	927
query37	125	70	73	70
query38	4037	3948	3955	3948
query39	1550	1453	1468	1453
query40	204	100	96	96
query41	48	44	44	44
query42	112	95	98	95
query43	532	516	481	481
query44	1165	807	823	807
query45	182	170	163	163
query46	1126	736	706	706
query47	1995	1942	1892	1892
query48	464	384	372	372
query49	705	381	368	368
query50	841	409	406	406
query51	7338	7227	7008	7008
query52	98	91	97	91
query53	247	183	178	178
query54	532	444	430	430
query55	73	74	74	74
query56	247	236	236	236
query57	1200	1144	1110	1110
query58	215	193	199	193
query59	3121	2975	3025	2975
query60	272	237	256	237
query61	116	98	96	96
query62	750	646	636	636
query63	211	184	178	178
query64	1615	624	604	604
query65	3224	3117	3310	3117
query66	633	285	288	285
query67	15989	15260	15252	15252
query68	4512	524	551	524
query69	403	258	249	249
query70	1173	1090	1148	1090
query71	316	242	259	242
query72	6460	4005	3929	3929
query73	751	334	335	334
query74	9708	8923	8884	8884
query75	3350	2625	2686	2625
query76	2069	1059	1030	1030
query77	465	266	263	263
query78	10714	9738	9478	9478
query79	7412	573	588	573
query80	2026	434	422	422
query81	558	244	240	240
query82	1125	121	113	113
query83	254	143	140	140
query84	288	82	78	78
query85	1826	358	353	353
query86	495	300	306	300
query87	4418	4259	4302	4259
query88	5751	2344	2359	2344
query89	416	289	292	289
query90	2086	176	185	176
query91	219	140	143	140
query92	65	47	46	46
query93	6180	526	534	526
query94	1052	274	281	274
query95	336	247	247	247
query96	630	271	266	266
query97	3331	3089	3139	3089
query98	214	193	192	192
query99	1878	1297	1288	1288
Total cold run time: 339281 ms
Total hot run time: 194312 ms

@dataroaring dataroaring merged commit 6da8694 into branch-3.0 Nov 26, 2024
22 of 24 checks passed
@github-actions github-actions bot deleted the auto-pick-43954-branch-3.0 branch November 26, 2024 11:18
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