Skip to content

Conversation

@924060929
Copy link
Contributor

What problem does this PR solve?

add ut for sql cache/sorted partition cache

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 Sep 1, 2025

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?

@924060929
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17629	5291	5115	5115
q2	1991	346	209	209
q3	10210	1320	725	725
q4	10247	1009	535	535
q5	7527	2442	2310	2310
q6	185	173	134	134
q7	923	738	636	636
q8	9352	1343	1091	1091
q9	6951	5119	5162	5119
q10	6896	2388	1981	1981
q11	479	316	279	279
q12	350	364	227	227
q13	17773	3642	3008	3008
q14	255	244	255	244
q15	548	493	486	486
q16	446	440	383	383
q17	589	856	375	375
q18	7523	7117	7008	7008
q19	1096	969	568	568
q20	359	347	235	235
q21	3951	2628	2350	2350
q22	1079	1056	1005	1005
Total cold run time: 106359 ms
Total hot run time: 34023 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5195	5193	5113	5113
q2	255	333	232	232
q3	2188	2678	2309	2309
q4	1412	1787	1325	1325
q5	4202	4324	4595	4324
q6	221	174	132	132
q7	2060	1977	1832	1832
q8	2645	2579	2589	2579
q9	7482	7620	7350	7350
q10	3102	3346	2881	2881
q11	577	519	494	494
q12	700	813	668	668
q13	3511	3900	3440	3440
q14	277	314	276	276
q15	524	490	505	490
q16	510	506	445	445
q17	1173	1565	1429	1429
q18	7905	7771	7598	7598
q19	826	876	940	876
q20	2003	2077	1932	1932
q21	5043	4551	4593	4551
q22	1075	1046	989	989
Total cold run time: 52886 ms
Total hot run time: 51265 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 186447 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 f9b95ec716cf84113bf6940b2c16e817e84d982a, data reload: false

query1	1045	438	425	425
query2	6567	1766	1827	1766
query3	6760	228	230	228
query4	26110	24025	23026	23026
query5	5110	661	508	508
query6	343	240	216	216
query7	4646	524	305	305
query8	305	251	256	251
query9	8634	2955	2925	2925
query10	546	347	315	315
query11	15532	15040	14784	14784
query12	175	129	116	116
query13	1672	571	434	434
query14	9018	5786	5831	5786
query15	213	202	171	171
query16	7223	681	504	504
query17	1190	748	613	613
query18	2015	443	334	334
query19	204	197	168	168
query20	143	132	129	129
query21	226	129	117	117
query22	4185	4193	4067	4067
query23	33929	32958	33296	32958
query24	8219	2431	2432	2431
query25	617	556	476	476
query26	1250	291	178	178
query27	2707	514	357	357
query28	4390	2312	2284	2284
query29	854	620	516	516
query30	297	236	202	202
query31	902	801	738	738
query32	96	85	78	78
query33	591	412	366	366
query34	794	860	519	519
query35	831	840	755	755
query36	960	1035	977	977
query37	137	110	92	92
query38	4050	3977	3992	3977
query39	1507	1441	1405	1405
query40	224	136	125	125
query41	71	62	65	62
query42	132	116	115	115
query43	529	507	498	498
query44	1358	881	870	870
query45	186	176	172	172
query46	888	1027	653	653
query47	1767	1811	1699	1699
query48	411	430	332	332
query49	771	513	421	421
query50	641	743	400	400
query51	4190	4152	4103	4103
query52	120	115	105	105
query53	253	270	201	201
query54	617	604	545	545
query55	95	91	91	91
query56	340	336	332	332
query57	1202	1196	1166	1166
query58	295	284	281	281
query59	2642	2683	2646	2646
query60	368	356	341	341
query61	169	162	159	159
query62	798	734	677	677
query63	241	192	194	192
query64	4509	1141	821	821
query65	4312	4210	4218	4210
query66	1165	438	363	363
query67	15445	15369	14994	14994
query68	8592	930	589	589
query69	488	332	294	294
query70	1327	1182	1151	1151
query71	469	352	343	343
query72	5903	4936	4906	4906
query73	727	582	365	365
query74	9090	9071	8700	8700
query75	3994	3091	2641	2641
query76	3744	1196	767	767
query77	810	415	352	352
query78	9507	9656	8795	8795
query79	2277	853	601	601
query80	670	586	518	518
query81	483	275	225	225
query82	460	144	109	109
query83	289	268	254	254
query84	303	113	96	96
query85	852	538	436	436
query86	343	318	318	318
query87	4254	4251	4212	4212
query88	2867	2264	2286	2264
query89	405	342	302	302
query90	2044	226	235	226
query91	161	158	137	137
query92	89	78	73	73
query93	1129	1002	649	649
query94	699	435	331	331
query95	412	330	329	329
query96	490	602	282	282
query97	2642	2723	2590	2590
query98	240	218	215	215
query99	1421	1416	1306	1306
Total cold run time: 274913 ms
Total hot run time: 186447 ms

@doris-robot
Copy link

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

query1	0.06	0.05	0.05
query2	0.10	0.06	0.06
query3	0.25	0.08	0.08
query4	1.62	0.11	0.12
query5	0.43	0.43	0.43
query6	1.18	0.64	0.68
query7	0.03	0.02	0.02
query8	0.05	0.05	0.05
query9	0.59	0.54	0.51
query10	0.61	0.58	0.58
query11	0.17	0.11	0.12
query12	0.15	0.12	0.12
query13	0.63	0.62	0.62
query14	0.79	0.84	0.84
query15	0.88	0.87	0.87
query16	0.39	0.43	0.40
query17	1.08	1.06	1.03
query18	0.22	0.20	0.20
query19	2.02	1.81	1.82
query20	0.02	0.01	0.01
query21	15.41	0.91	0.60
query22	0.80	1.17	0.84
query23	14.73	1.36	0.63
query24	6.72	1.50	0.42
query25	0.50	0.19	0.07
query26	0.60	0.16	0.13
query27	0.06	0.06	0.06
query28	10.15	1.00	0.43
query29	12.53	3.87	3.21
query30	3.09	3.01	2.95
query31	2.81	0.58	0.38
query32	3.24	0.55	0.47
query33	3.18	3.09	3.25
query34	15.96	5.46	4.86
query35	4.91	4.92	4.93
query36	0.70	0.52	0.49
query37	0.10	0.07	0.07
query38	0.06	0.05	0.05
query39	0.04	0.03	0.03
query40	0.18	0.15	0.14
query41	0.09	0.04	0.03
query42	0.03	0.03	0.03
query43	0.04	0.03	0.04
Total cold run time: 107.2 s
Total hot run time: 32.47 s

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

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

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Sep 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2025

PR approved by anyone and no changes requested.

@924060929 924060929 merged commit 5144826 into apache:master Sep 1, 2025
28 of 30 checks passed
@924060929 924060929 deleted the master-add-ut branch September 1, 2025 08:19
github-actions bot pushed a commit that referenced this pull request Sep 1, 2025
morrySnow pushed a commit that referenced this pull request Sep 4, 2025
@morrySnow morrySnow mentioned this pull request Sep 22, 2025
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/3.1.1-merged reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants