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

[fix](test) fix case test_index_io_context #47290

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zzzxl1993
Copy link
Contributor

@zzzxl1993 zzzxl1993 commented Jan 21, 2025

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

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

@zzzxl1993
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17562	5443	5359	5359
q2	2056	306	168	168
q3	10426	1212	721	721
q4	10258	966	542	542
q5	8307	2388	2188	2188
q6	195	162	132	132
q7	895	757	602	602
q8	9241	1319	1143	1143
q9	5110	4841	4925	4841
q10	6829	2337	1875	1875
q11	470	280	258	258
q12	340	348	211	211
q13	17788	3654	3204	3204
q14	229	219	209	209
q15	520	473	468	468
q16	629	618	574	574
q17	567	858	324	324
q18	6801	6314	6400	6314
q19	1647	942	552	552
q20	309	322	192	192
q21	2827	2186	2004	2004
q22	368	335	309	309
Total cold run time: 103374 ms
Total hot run time: 32190 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5519	5580	5490	5490
q2	246	340	232	232
q3	2229	2653	2343	2343
q4	1418	1818	1393	1393
q5	4310	4707	4607	4607
q6	164	157	130	130
q7	2007	1938	1846	1846
q8	2630	2786	2700	2700
q9	7330	7248	7178	7178
q10	3033	3280	2778	2778
q11	566	510	484	484
q12	656	716	595	595
q13	3536	4021	3319	3319
q14	278	308	289	289
q15	527	490	467	467
q16	642	680	651	651
q17	1199	1730	1276	1276
q18	7565	7367	7370	7367
q19	790	1097	1120	1097
q20	2010	2052	1896	1896
q21	5807	5288	4848	4848
q22	575	573	572	572
Total cold run time: 53037 ms
Total hot run time: 51558 ms

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 41.57% (10834/26060)
Line Coverage: 31.96% (91554/286478)
Region Coverage: 31.10% (46898/150815)
Branch Coverage: 27.19% (23755/87380)
Coverage Report: http://coverage.selectdb-in.cc/coverage/1876daffab5a5bdbca79a4cfe16e7f4b1025a03e_1876daffab5a5bdbca79a4cfe16e7f4b1025a03e/report/index.html

@doris-robot
Copy link

TPC-DS: Total hot run time: 194160 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 1876daffab5a5bdbca79a4cfe16e7f4b1025a03e, data reload: false

query1	1305	946	925	925
query2	6341	2091	2054	2054
query3	11098	4562	4647	4562
query4	61475	29034	23218	23218
query5	5522	598	457	457
query6	401	193	177	177
query7	5550	491	303	303
query8	325	252	220	220
query9	7986	2616	2589	2589
query10	448	313	263	263
query11	17385	15141	15449	15141
query12	163	109	116	109
query13	1397	568	419	419
query14	11570	6863	6939	6863
query15	201	214	184	184
query16	7262	640	452	452
query17	1144	760	614	614
query18	1923	410	332	332
query19	209	205	164	164
query20	124	166	106	106
query21	212	120	102	102
query22	4534	4373	4627	4373
query23	34224	33717	33503	33503
query24	5663	2374	2442	2374
query25	491	459	399	399
query26	653	272	165	165
query27	1843	484	337	337
query28	4284	2502	2428	2428
query29	553	560	436	436
query30	225	195	156	156
query31	955	882	833	833
query32	71	62	59	59
query33	416	356	331	331
query34	773	876	506	506
query35	829	858	778	778
query36	1027	1037	958	958
query37	123	107	79	79
query38	4192	4332	4166	4166
query39	1498	1436	1438	1436
query40	208	114	98	98
query41	51	95	47	47
query42	120	100	100	100
query43	532	522	495	495
query44	1392	857	840	840
query45	181	173	169	169
query46	890	1084	655	655
query47	1953	1913	1830	1830
query48	387	420	327	327
query49	730	475	401	401
query50	681	682	417	417
query51	7061	7109	7065	7065
query52	97	99	95	95
query53	231	262	200	200
query54	488	504	422	422
query55	89	78	83	78
query56	269	286	279	279
query57	1226	1171	1148	1148
query58	245	245	234	234
query59	3130	3179	2923	2923
query60	271	269	260	260
query61	119	114	111	111
query62	709	718	622	622
query63	228	190	184	184
query64	1328	1051	662	662
query65	3254	3142	3161	3142
query66	662	406	289	289
query67	15853	15675	15399	15399
query68	5722	853	534	534
query69	509	292	262	262
query70	1181	1152	1134	1134
query71	449	289	277	277
query72	6311	3995	3848	3848
query73	788	748	350	350
query74	10105	8864	8756	8756
query75	3211	3130	2653	2653
query76	3833	1181	780	780
query77	477	387	266	266
query78	10030	10063	9369	9369
query79	3140	816	581	581
query80	665	543	435	435
query81	493	279	234	234
query82	413	148	122	122
query83	167	175	149	149
query84	289	93	89	89
query85	753	348	301	301
query86	383	313	289	289
query87	4364	4639	4380	4380
query88	4589	2128	2119	2119
query89	399	327	297	297
query90	1625	188	191	188
query91	128	134	107	107
query92	64	57	53	53
query93	2904	870	533	533
query94	722	403	294	294
query95	339	264	256	256
query96	494	614	284	284
query97	2852	2869	2710	2710
query98	226	203	189	189
query99	1290	1364	1258	1258
Total cold run time: 315695 ms
Total hot run time: 194160 ms

@doris-robot
Copy link

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

query1	0.04	0.04	0.03
query2	0.08	0.04	0.04
query3	0.24	0.08	0.06
query4	1.62	0.10	0.10
query5	0.42	0.41	0.43
query6	1.16	0.66	0.64
query7	0.03	0.02	0.02
query8	0.04	0.03	0.04
query9	0.58	0.50	0.51
query10	0.55	0.56	0.55
query11	0.15	0.11	0.11
query12	0.13	0.11	0.11
query13	0.60	0.60	0.60
query14	2.70	2.84	2.82
query15	0.90	0.83	0.84
query16	0.36	0.39	0.38
query17	1.07	1.05	0.99
query18	0.23	0.20	0.21
query19	1.94	1.82	1.99
query20	0.01	0.01	0.01
query21	15.36	0.92	0.57
query22	0.80	0.86	0.70
query23	15.16	1.50	0.55
query24	3.04	1.60	1.04
query25	0.20	0.10	0.08
query26	0.28	0.16	0.13
query27	0.05	0.06	0.05
query28	13.88	1.05	0.44
query29	12.58	3.96	3.26
query30	0.25	0.10	0.06
query31	2.83	0.62	0.38
query32	3.23	0.56	0.47
query33	3.04	2.98	3.05
query34	16.69	5.36	4.48
query35	4.56	4.53	4.56
query36	0.64	0.52	0.50
query37	0.10	0.06	0.07
query38	0.05	0.03	0.04
query39	0.04	0.02	0.03
query40	0.17	0.14	0.13
query41	0.09	0.02	0.02
query42	0.03	0.02	0.03
query43	0.04	0.02	0.02
Total cold run time: 105.96 s
Total hot run time: 30.77 s

Copy link
Member

@airborne12 airborne12 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

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

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Jan 22, 2025
Copy link
Contributor

PR approved by anyone and no changes requested.

Copy link
Collaborator

@shuke987 shuke987 left a comment

Choose a reason for hiding this comment

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

LGTM

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.0.x reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants