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](jdbc catalog) Fix query errors without jdbc pool default value on only BE upgrade #32618

Merged
merged 1 commit into from Mar 22, 2024

Conversation

zy-kkk
Copy link
Member

@zy-kkk zy-kkk commented Mar 21, 2024

Proposed changes

Issue Number: close #xxx

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@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

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@zy-kkk
Copy link
Member Author

zy-kkk commented Mar 21, 2024

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17647	4231	4085	4085
q2	2020	153	147	147
q3	10585	1130	1160	1130
q4	10227	782	791	782
q5	7475	3014	2943	2943
q6	206	125	119	119
q7	1005	567	550	550
q8	9310	1991	1956	1956
q9	7091	6479	6366	6366
q10	8412	3261	3564	3261
q11	423	223	215	215
q12	408	195	187	187
q13	17790	2845	2825	2825
q14	248	199	206	199
q15	488	453	463	453
q16	471	359	355	355
q17	937	620	585	585
q18	7039	6477	6412	6412
q19	3221	1350	1429	1350
q20	536	244	245	244
q21	3561	2829	2926	2829
q22	351	321	286	286
Total cold run time: 109451 ms
Total hot run time: 37279 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4109	4075	4037	4037
q2	315	243	219	219
q3	2947	2892	2817	2817
q4	1825	1548	1532	1532
q5	5219	5305	5246	5246
q6	190	114	115	114
q7	2240	1890	1872	1872
q8	3153	3307	3289	3289
q9	8532	8504	8524	8504
q10	3670	3665	3634	3634
q11	539	436	435	435
q12	731	562	539	539
q13	16917	2859	2854	2854
q14	276	263	251	251
q15	494	450	439	439
q16	454	410	412	410
q17	1731	1500	1455	1455
q18	7298	7224	7114	7114
q19	1604	1448	1547	1448
q20	1900	1692	1685	1685
q21	4634	4758	4766	4758
q22	531	450	443	443
Total cold run time: 69309 ms
Total hot run time: 53095 ms

@doris-robot
Copy link

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

query1	934	364	346	346
query2	7507	2091	2014	2014
query3	6711	213	205	205
query4	31221	20662	20566	20566
query5	4403	409	414	409
query6	269	172	167	167
query7	4628	280	291	280
query8	224	161	172	161
query9	9863	2252	2239	2239
query10	458	241	243	241
query11	14580	14235	14217	14217
query12	140	95	86	86
query13	1622	419	419	419
query14	10868	10816	10350	10350
query15	262	195	206	195
query16	8116	256	254	254
query17	1955	582	513	513
query18	2075	273	265	265
query19	243	150	149	149
query20	92	84	80	80
query21	202	130	137	130
query22	4700	4490	4525	4490
query23	31665	31039	31063	31039
query24	10786	2869	2819	2819
query25	573	355	362	355
query26	928	148	145	145
query27	2415	333	339	333
query28	6881	1868	1833	1833
query29	861	655	616	616
query30	312	147	146	146
query31	941	722	744	722
query32	91	59	55	55
query33	770	340	252	252
query34	991	486	487	486
query35	837	605	595	595
query36	994	856	858	856
query37	107	78	78	78
query38	3531	3450	3484	3450
query39	1453	1362	1365	1362
query40	208	113	111	111
query41	48	48	48	48
query42	103	94	93	93
query43	495	445	459	445
query44	1257	714	707	707
query45	287	252	262	252
query46	1087	669	693	669
query47	1647	1614	1600	1600
query48	430	363	355	355
query49	1100	336	335	335
query50	754	364	366	364
query51	6734	6633	6572	6572
query52	108	94	88	88
query53	338	271	278	271
query54	313	242	268	242
query55	84	81	78	78
query56	263	236	241	236
query57	1063	1003	1010	1003
query58	228	207	208	207
query59	2835	2597	2595	2595
query60	282	247	264	247
query61	113	109	112	109
query62	592	393	404	393
query63	300	277	281	277
query64	5109	3871	3884	3871
query65	3033	2994	2993	2993
query66	868	348	348	348
query67	14859	14381	14320	14320
query68	8106	525	528	525
query69	618	387	368	368
query70	1222	1148	1174	1148
query71	525	288	288	288
query72	6672	2667	2503	2503
query73	730	309	312	309
query74	7024	6516	6575	6516
query75	4339	2807	2873	2807
query76	4975	906	1025	906
query77	669	279	253	253
query78	10240	9555	9620	9555
query79	12732	518	510	510
query80	1840	394	383	383
query81	507	209	211	209
query82	685	216	195	195
query83	213	146	142	142
query84	287	75	79	75
query85	1251	330	309	309
query86	406	317	306	306
query87	3735	3562	3531	3531
query88	5092	2283	2279	2279
query89	493	359	353	353
query90	1989	178	173	173
query91	160	152	130	130
query92	58	48	46	46
query93	7221	496	486	486
query94	1253	177	174	174
query95	429	327	326	326
query96	619	268	267	267
query97	3027	2842	2835	2835
query98	225	211	219	211
query99	1045	739	758	739
Total cold run time: 308134 ms
Total hot run time: 180604 ms

Copy link
Contributor

PR approved by anyone and no changes requested.

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 Mar 22, 2024
@zy-kkk zy-kkk merged commit 1fc91a3 into apache:master Mar 22, 2024
29 of 31 checks passed
@zy-kkk zy-kkk deleted the fix_jdbc_conn_pool_be_default branch March 22, 2024 02:57
yiguolei pushed a commit that referenced this pull request Mar 22, 2024
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/2.0.7-merged reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants