Skip to content

Conversation

@0AyanamiRei
Copy link
Contributor

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Similar to the content of #59743, certain information in the HTTP headers is hidden to avoid being printed in the logs.

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?

@0AyanamiRei
Copy link
Contributor Author

run buildall

@0AyanamiRei
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17622	4578	4277	4277
q2	2044	347	229	229
q3	10181	1292	717	717
q4	10189	781	303	303
q5	7539	2196	1891	1891
q6	193	177	145	145
q7	881	726	604	604
q8	9260	1357	1095	1095
q9	4664	4671	4617	4617
q10	6794	1930	1534	1534
q11	459	273	239	239
q12	346	383	219	219
q13	17952	4065	3264	3264
q14	245	249	228	228
q15	939	831	815	815
q16	698	681	632	632
q17	730	764	634	634
q18	6644	5837	5966	5837
q19	1233	1014	650	650
q20	521	512	395	395
q21	2654	1923	1809	1809
q22	335	289	265	265
Total cold run time: 102123 ms
Total hot run time: 30399 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4398	4332	4345	4332
q2	282	348	257	257
q3	2186	2785	2285	2285
q4	1422	1751	1284	1284
q5	4241	4202	4388	4202
q6	210	174	135	135
q7	1992	1783	1628	1628
q8	2666	2585	2471	2471
q9	7677	7636	7468	7468
q10	2860	3046	2598	2598
q11	543	486	436	436
q12	710	792	669	669
q13	3846	4391	3594	3594
q14	314	305	293	293
q15	882	840	809	809
q16	651	705	669	669
q17	1154	1290	1350	1290
q18	8500	7833	7974	7833
q19	922	873	896	873
q20	2030	2178	1973	1973
q21	4637	4237	4018	4018
q22	469	459	419	419
Total cold run time: 52592 ms
Total hot run time: 49536 ms

Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Feb 11, 2026
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@doris-robot
Copy link

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

query1	0.06	0.05	0.04
query2	0.09	0.05	0.04
query3	0.25	0.08	0.08
query4	1.61	0.12	0.11
query5	0.26	0.27	0.25
query6	1.16	0.67	0.67
query7	0.04	0.03	0.03
query8	0.05	0.03	0.04
query9	0.57	0.51	0.49
query10	0.55	0.54	0.55
query11	0.14	0.10	0.10
query12	0.14	0.11	0.10
query13	0.62	0.60	0.62
query14	1.07	1.06	1.05
query15	0.87	0.87	0.87
query16	0.41	0.41	0.40
query17	1.14	1.15	1.11
query18	0.23	0.21	0.21
query19	2.12	2.00	2.05
query20	0.02	0.01	0.02
query21	15.40	0.27	0.15
query22	5.18	0.06	0.05
query23	15.91	0.29	0.11
query24	2.31	0.66	0.27
query25	0.11	0.06	0.07
query26	0.13	0.14	0.14
query27	0.06	0.06	0.08
query28	4.16	1.13	0.97
query29	12.60	4.00	3.19
query30	0.27	0.13	0.12
query31	2.80	0.64	0.41
query32	3.25	0.60	0.49
query33	3.21	3.23	3.33
query34	16.40	5.40	4.72
query35	4.77	4.78	4.78
query36	0.65	0.50	0.50
query37	0.13	0.08	0.07
query38	0.08	0.04	0.05
query39	0.05	0.02	0.02
query40	0.20	0.15	0.16
query41	0.09	0.04	0.02
query42	0.05	0.03	0.03
query43	0.04	0.04	0.03
Total cold run time: 99.25 s
Total hot run time: 28.34 s

@hello-stephen
Copy link
Contributor

BE UT Coverage Report

Increment line coverage 85.71% (6/7) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 52.72% (19456/36904)
Line Coverage 36.23% (181259/500343)
Region Coverage 32.60% (140719/431697)
Branch Coverage 33.63% (60965/181272)

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. dev/4.0.x reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants