Skip to content

[fix](cloud) reduce warning log noise in peer read path#60889

Open
deardeng wants to merge 2 commits intoapache:masterfrom
deardeng:reduce-peer-warning-log-noise
Open

[fix](cloud) reduce warning log noise in peer read path#60889
deardeng wants to merge 2 commits intoapache:masterfrom
deardeng:reduce-peer-warning-log-noise

Conversation

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

@Thearas
Copy link
Contributor

Thearas commented Feb 27, 2026

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?

@deardeng
Copy link
Contributor Author

run buildall

@deardeng
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
============================================
q1	17608	4548	4312	4312
q2	q3	10659	796	528	528
q4	4691	354	257	257
q5	7557	1199	1035	1035
q6	182	178	151	151
q7	796	860	684	684
q8	9299	1502	1416	1416
q9	4910	4740	4690	4690
q10	6833	1895	1645	1645
q11	456	266	244	244
q12	701	563	463	463
q13	17788	4238	3451	3451
q14	222	230	206	206
q15	955	790	790	790
q16	743	733	662	662
q17	710	945	406	406
q18	5927	5475	5232	5232
q19	1115	968	629	629
q20	524	498	395	395
q21	4437	1916	1480	1480
q22	341	288	247	247
Total cold run time: 96454 ms
Total hot run time: 28923 ms

----- Round 2, with runtime_filter_mode=off -----
============================================
q1	4426	4349	4372	4349
q2	q3	1774	2171	1716	1716
q4	884	1160	786	786
q5	4013	4330	4296	4296
q6	179	172	146	146
q7	1724	1600	1462	1462
q8	2405	2652	2515	2515
q9	7303	7591	7561	7561
q10	2711	2838	2410	2410
q11	525	438	419	419
q12	517	603	444	444
q13	3955	4475	3714	3714
q14	294	302	292	292
q15	867	802	793	793
q16	726	753	711	711
q17	1162	1527	1284	1284
q18	7056	6709	6748	6709
q19	941	925	915	915
q20	2079	2191	1987	1987
q21	4170	3502	3490	3490
q22	504	419	383	383
Total cold run time: 48215 ms
Total hot run time: 46382 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 183974 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 87a2bef0c41d08f21ea281a44358dbf61b8103d4, data reload: false

query5	5130	672	507	507
query6	329	223	205	205
query7	4211	483	280	280
query8	352	259	258	258
query9	8760	2756	2759	2756
query10	557	382	333	333
query11	16985	17524	17345	17345
query12	208	139	129	129
query13	1436	503	352	352
query14	6985	3678	3150	3150
query14_1	2977	2984	3021	2984
query15	211	213	191	191
query16	1040	504	458	458
query17	1116	806	658	658
query18	3025	482	364	364
query19	230	227	192	192
query20	148	135	138	135
query21	261	141	119	119
query22	5478	4952	4853	4853
query23	17236	16740	16461	16461
query23_1	16578	16631	16618	16618
query24	7275	1606	1222	1222
query24_1	1233	1206	1256	1206
query25	538	456	394	394
query26	1248	263	149	149
query27	2730	487	292	292
query28	4449	1864	1879	1864
query29	787	572	509	509
query30	308	243	210	210
query31	872	729	637	637
query32	76	74	66	66
query33	500	344	289	289
query34	930	916	569	569
query35	645	689	588	588
query36	1056	1075	968	968
query37	134	94	84	84
query38	2925	2928	2888	2888
query39	884	854	842	842
query39_1	813	829	822	822
query40	232	162	135	135
query41	79	58	58	58
query42	104	103	103	103
query43	380	382	354	354
query44	
query45	206	187	186	186
query46	900	993	616	616
query47	2121	2112	2029	2029
query48	311	322	234	234
query49	623	461	383	383
query50	699	283	217	217
query51	4063	4055	3991	3991
query52	108	111	102	102
query53	294	338	292	292
query54	296	281	261	261
query55	97	84	83	83
query56	317	302	303	302
query57	1344	1332	1270	1270
query58	293	272	291	272
query59	2589	2746	2522	2522
query60	341	342	327	327
query61	150	146	149	146
query62	624	593	537	537
query63	318	279	281	279
query64	4837	1275	981	981
query65	
query66	1370	449	349	349
query67	16406	16477	16505	16477
query68	
query69	388	312	282	282
query70	994	959	967	959
query71	355	322	313	313
query72	2805	2695	2372	2372
query73	553	584	327	327
query74	9935	9941	9758	9758
query75	2839	2756	2463	2463
query76	2298	1040	685	685
query77	374	417	312	312
query78	11201	11389	10728	10728
query79	1158	831	611	611
query80	1382	612	532	532
query81	556	282	266	266
query82	1010	150	140	140
query83	345	263	247	247
query84	251	116	101	101
query85	887	465	424	424
query86	430	308	326	308
query87	3110	3103	2993	2993
query88	3591	2666	2643	2643
query89	430	379	351	351
query90	1972	181	172	172
query91	169	150	130	130
query92	77	73	73	73
query93	981	834	507	507
query94	629	316	283	283
query95	583	397	335	335
query96	644	532	226	226
query97	2517	2488	2382	2382
query98	230	234	215	215
query99	1032	982	927	927
Total cold run time: 255488 ms
Total hot run time: 183974 ms

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 0.00% (0/8) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.53% (19579/37275)
Line Coverage 36.14% (182697/505481)
Region Coverage 32.48% (141864/436763)
Branch Coverage 33.45% (61520/183911)

@hello-stephen
Copy link
Contributor

BE Regression && UT Coverage Report

Increment line coverage 75.00% (6/8) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 71.54% (26119/36508)
Line Coverage 54.29% (273635/503987)
Region Coverage 51.67% (227835/440939)
Branch Coverage 53.07% (97912/184503)

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.

4 participants