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

branch-3.0: [fix](index) Fix create index/index def to sql #44392 #44493

Open
wants to merge 2 commits into
base: branch-3.0
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

Cherry-picked from #44392

Fix toSql for CreateIndexClause with alter
@doris-robot
Copy link

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?

@dataroaring dataroaring reopened this Nov 22, 2024
@doris-robot
Copy link

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17583	7381	7237	7237
q2	2055	159	154	154
q3	10713	1050	1169	1050
q4	10545	733	692	692
q5	7784	2789	2790	2789
q6	233	147	147	147
q7	971	598	594	594
q8	9562	1940	1934	1934
q9	7694	6354	6447	6354
q10	6985	2270	2309	2270
q11	456	257	259	257
q12	413	218	210	210
q13	17765	3002	2949	2949
q14	232	202	205	202
q15	550	527	532	527
q16	671	593	587	587
q17	961	593	653	593
q18	7093	6579	6467	6467
q19	1383	976	954	954
q20	476	193	199	193
q21	4064	3221	3125	3125
q22	1103	1004	992	992
Total cold run time: 109292 ms
Total hot run time: 40277 ms

----- Round 2, with runtime_filter_mode=off -----
q1	7236	7193	7310	7193
q2	321	223	231	223
q3	2797	2859	2809	2809
q4	1945	1733	1813	1733
q5	5648	5678	5690	5678
q6	220	139	146	139
q7	2240	1762	1771	1762
q8	3314	3528	3513	3513
q9	8740	8824	8813	8813
q10	3544	3531	3511	3511
q11	612	497	499	497
q12	797	607	590	590
q13	16491	3143	3147	3143
q14	292	278	267	267
q15	552	510	520	510
q16	697	673	662	662
q17	1829	1630	1605	1605
q18	8069	7955	7692	7692
q19	6007	1573	1493	1493
q20	2036	1904	1844	1844
q21	5281	5265	5202	5202
q22	1150	1060	1020	1020
Total cold run time: 79818 ms
Total hot run time: 59899 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 196242 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 76025f802f33f329be5e904c34ab5d9bb3ffe364, data reload: false

query1	1246	909	886	886
query2	6281	2036	1972	1972
query3	10915	3951	3836	3836
query4	67875	30339	23445	23445
query5	5173	459	446	446
query6	416	177	171	171
query7	5623	312	308	308
query8	367	285	299	285
query9	9126	2682	2676	2676
query10	478	272	259	259
query11	17602	15261	15635	15261
query12	163	102	110	102
query13	1537	434	425	425
query14	10635	7197	7222	7197
query15	222	188	179	179
query16	7616	528	524	524
query17	1115	592	635	592
query18	1981	332	313	313
query19	209	166	162	162
query20	126	114	120	114
query21	206	108	106	106
query22	4580	4484	4307	4307
query23	34878	33902	34288	33902
query24	6136	3003	2877	2877
query25	536	431	415	415
query26	666	168	167	167
query27	1718	293	308	293
query28	4271	2530	2557	2530
query29	704	451	454	451
query30	238	165	164	164
query31	1009	837	837	837
query32	74	57	55	55
query33	469	282	276	276
query34	916	506	508	506
query35	823	742	764	742
query36	1076	983	943	943
query37	121	75	73	73
query38	4060	3967	4012	3967
query39	1542	1462	1453	1453
query40	209	102	100	100
query41	55	51	53	51
query42	105	103	99	99
query43	536	498	501	498
query44	1162	819	810	810
query45	196	171	166	166
query46	1139	726	720	720
query47	1977	1890	1892	1890
query48	489	375	378	375
query49	764	419	416	416
query50	834	421	425	421
query51	7425	7153	7074	7074
query52	105	94	92	92
query53	247	184	187	184
query54	556	439	450	439
query55	78	76	76	76
query56	258	259	239	239
query57	1204	1086	1093	1086
query58	242	212	217	212
query59	3283	2984	2814	2814
query60	276	262	259	259
query61	128	122	121	121
query62	780	643	681	643
query63	212	191	197	191
query64	1602	768	736	736
query65	3240	3222	3157	3157
query66	723	317	323	317
query67	15790	15323	15393	15323
query68	4451	556	545	545
query69	420	256	255	255
query70	1167	1116	1106	1106
query71	348	254	252	252
query72	6548	4232	4243	4232
query73	751	344	347	344
query74	9877	8833	8932	8833
query75	3337	2632	2687	2632
query76	1863	1109	1081	1081
query77	502	284	281	281
query78	10800	9632	9582	9582
query79	7978	596	599	596
query80	2376	464	459	459
query81	567	249	249	249
query82	1322	121	129	121
query83	300	148	148	148
query84	331	91	79	79
query85	1524	361	359	359
query86	479	290	296	290
query87	4524	4112	4225	4112
query88	5492	2356	2392	2356
query89	423	292	289	289
query90	2022	181	179	179
query91	182	153	150	150
query92	74	50	50	50
query93	6687	542	541	541
query94	1087	308	296	296
query95	359	259	267	259
query96	633	272	281	272
query97	3401	3096	3174	3096
query98	213	192	192	192
query99	1634	1280	1320	1280
Total cold run time: 338016 ms
Total hot run time: 196242 ms

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

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