Skip to content

Conversation

@englefly
Copy link
Contributor

What problem does this PR solve?

pick part of #49947
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

@englefly englefly requested a review from morrySnow as a code owner June 24, 2025 02:53
@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?

@morrySnow
Copy link
Contributor

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17580	7469	6611	6611
q2	2056	172	184	172
q3	10539	1091	1133	1091
q4	10522	766	772	766
q5	7790	2877	2913	2877
q6	217	138	138	138
q7	1010	639	628	628
q8	9353	1987	2072	1987
q9	6683	6362	6451	6362
q10	7056	2263	2286	2263
q11	478	272	266	266
q12	404	218	217	217
q13	17817	3011	3050	3011
q14	229	214	223	214
q15	523	485	472	472
q16	481	383	386	383
q17	1007	673	557	557
q18	7390	6725	6720	6720
q19	1345	1003	910	910
q20	495	205	200	200
q21	3956	3265	3252	3252
q22	1107	1013	1009	1009
Total cold run time: 108038 ms
Total hot run time: 40106 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6652	6637	6577	6577
q2	329	231	238	231
q3	2927	2783	2949	2783
q4	2058	1791	1816	1791
q5	5739	5756	5782	5756
q6	211	126	128	126
q7	2245	1831	1812	1812
q8	3445	3574	3556	3556
q9	8895	8868	9000	8868
q10	3562	3535	3513	3513
q11	599	493	495	493
q12	840	616	626	616
q13	7968	3154	3177	3154
q14	298	254	273	254
q15	521	472	462	462
q16	512	441	449	441
q17	1847	1653	1597	1597
q18	8376	7976	7684	7684
q19	1716	1628	1534	1534
q20	2081	1819	1804	1804
q21	5267	5061	5001	5001
q22	1147	1036	1055	1036
Total cold run time: 67235 ms
Total hot run time: 59089 ms

@morrySnow morrySnow merged commit bdf4a32 into apache:branch-3.1 Jun 24, 2025
20 of 22 checks passed
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