Skip to content

Conversation

@zy-kkk
Copy link
Member

@zy-kkk zy-kkk commented Aug 14, 2024

In previous versions, we used a method based on JDBC 4.2 to read data, so it was equivalent to abandoning support for ojdbc6. However, we recently found that a large number of users still use Oracle version 11g, which will have some unexpected compatibility issues when using ojdbc8 to connect. Therefore, I use version verification to make it compatible with both ojdbc6 and ojdbc8, so that good compatibility can be obtained through ojdbc6, and better reading efficiency can be obtained through ojdbc8.

@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 Aug 14, 2024

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17643	4329	4250	4250
q2	2019	187	187	187
q3	11748	970	1101	970
q4	10511	738	771	738
q5	7766	2801	2761	2761
q6	223	137	136	136
q7	949	590	586	586
q8	9308	2045	2044	2044
q9	7179	6513	6535	6513
q10	7020	2244	2235	2235
q11	443	243	243	243
q12	396	224	225	224
q13	18759	2986	3005	2986
q14	286	232	236	232
q15	535	478	487	478
q16	495	405	381	381
q17	963	741	707	707
q18	7259	6874	6819	6819
q19	7260	1070	943	943
q20	790	340	336	336
q21	3967	2949	2812	2812
q22	1091	1008	1008	1008
Total cold run time: 116610 ms
Total hot run time: 37589 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4426	4289	4253	4253
q2	372	275	265	265
q3	2837	2612	2599	2599
q4	1899	1703	1693	1693
q5	5661	5773	5585	5585
q6	221	132	137	132
q7	2176	1762	1762	1762
q8	3312	3455	3445	3445
q9	8708	8698	8742	8698
q10	3578	3260	3276	3260
q11	592	523	520	520
q12	808	612	604	604
q13	16153	3152	3128	3128
q14	324	288	295	288
q15	521	490	501	490
q16	492	446	447	446
q17	1817	1558	1501	1501
q18	8062	7926	7996	7926
q19	1801	1708	1618	1618
q20	2140	1909	1902	1902
q21	5616	5407	5344	5344
q22	1121	1022	1041	1022
Total cold run time: 72637 ms
Total hot run time: 56481 ms

Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@doris-robot
Copy link

TPC-DS: Total hot run time: 189073 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 27139502490b745ea24b2f8bd31671dafd61b6a6, data reload: false

query1	1243	902	865	865
query2	6438	1930	1878	1878
query3	10599	3771	3797	3771
query4	59292	23930	23382	23382
query5	5742	502	486	486
query6	497	156	152	152
query7	6509	296	297	296
query8	302	210	202	202
query9	8857	2442	2412	2412
query10	496	256	263	256
query11	18035	15079	15150	15079
query12	162	100	102	100
query13	1592	389	374	374
query14	11764	6784	6731	6731
query15	243	185	174	174
query16	7608	529	495	495
query17	1154	583	573	573
query18	2093	311	303	303
query19	291	165	151	151
query20	127	123	116	116
query21	217	114	110	110
query22	4396	4323	4250	4250
query23	34075	33632	33007	33007
query24	5491	2954	2886	2886
query25	543	408	394	394
query26	687	160	155	155
query27	1816	267	274	267
query28	3790	2043	2036	2036
query29	716	418	409	409
query30	252	152	148	148
query31	902	752	753	752
query32	82	52	54	52
query33	506	283	296	283
query34	852	451	456	451
query35	841	715	725	715
query36	1042	927	899	899
query37	134	82	79	79
query38	3974	3866	3904	3866
query39	1465	1375	1389	1375
query40	207	116	118	116
query41	48	48	47	47
query42	121	94	96	94
query43	503	458	461	458
query44	1054	729	741	729
query45	198	162	164	162
query46	1090	776	741	741
query47	1830	1733	1769	1733
query48	366	286	294	286
query49	766	423	420	420
query50	795	396	410	396
query51	6742	6701	6563	6563
query52	105	90	91	90
query53	258	186	182	182
query54	572	455	555	455
query55	73	73	71	71
query56	251	242	249	242
query57	1129	1033	1032	1032
query58	216	240	242	240
query59	2902	2748	2806	2748
query60	307	270	258	258
query61	94	96	95	95
query62	751	639	670	639
query63	214	179	179	179
query64	3314	1722	1712	1712
query65	3210	3141	3155	3141
query66	700	330	329	329
query67	15389	14774	14844	14774
query68	8658	554	566	554
query69	606	389	268	268
query70	1504	1180	1159	1159
query71	536	276	269	269
query72	6520	2240	1936	1936
query73	2579	314	316	314
query74	9549	8772	8790	8772
query75	4181	2664	2708	2664
query76	4700	1015	973	973
query77	727	315	359	315
query78	9632	9143	8944	8944
query79	10347	524	522	522
query80	1510	512	486	486
query81	582	225	223	223
query82	658	138	137	137
query83	224	142	146	142
query84	268	72	79	72
query85	1031	264	297	264
query86	333	290	281	281
query87	4379	4124	4213	4124
query88	4558	2309	2318	2309
query89	506	295	285	285
query90	2298	193	189	189
query91	120	93	94	93
query92	63	49	49	49
query93	5621	545	539	539
query94	1064	287	285	285
query95	352	254	256	254
query96	616	272	269	269
query97	3203	3061	3102	3061
query98	220	208	196	196
query99	1484	1263	1263	1263
Total cold run time: 336659 ms
Total hot run time: 189073 ms

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Aug 14, 2024
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@doris-robot
Copy link

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

query1	0.04	0.04	0.04
query2	0.08	0.04	0.04
query3	0.22	0.05	0.05
query4	1.68	0.07	0.07
query5	0.51	0.49	0.47
query6	1.13	0.72	0.73
query7	0.01	0.01	0.02
query8	0.05	0.04	0.04
query9	0.54	0.48	0.49
query10	0.54	0.55	0.53
query11	0.15	0.12	0.12
query12	0.15	0.11	0.12
query13	0.62	0.60	0.59
query14	0.78	0.78	0.78
query15	0.87	0.82	0.82
query16	0.36	0.35	0.38
query17	1.07	0.99	0.99
query18	0.22	0.22	0.23
query19	1.82	1.77	1.68
query20	0.01	0.01	0.01
query21	15.40	0.74	0.65
query22	4.58	8.20	1.19
query23	18.28	1.32	1.32
query24	2.12	0.23	0.21
query25	0.15	0.09	0.07
query26	0.29	0.22	0.21
query27	0.46	0.23	0.23
query28	13.36	1.01	1.00
query29	12.61	3.42	3.36
query30	0.23	0.06	0.04
query31	2.88	0.40	0.40
query32	3.26	0.49	0.49
query33	2.97	2.98	2.96
query34	16.89	4.35	4.44
query35	4.41	4.43	4.42
query36	0.66	0.46	0.47
query37	0.19	0.16	0.15
query38	0.16	0.14	0.15
query39	0.04	0.04	0.03
query40	0.15	0.13	0.12
query41	0.09	0.05	0.04
query42	0.06	0.04	0.05
query43	0.05	0.04	0.04
Total cold run time: 110.14 s
Total hot run time: 30.04 s

@morningman morningman merged commit 0cba9ba into apache:master Aug 14, 2024
@zy-kkk zy-kkk deleted the ojdbc6 branch August 14, 2024 13:53
zy-kkk added a commit to zy-kkk/doris that referenced this pull request Aug 15, 2024
…check (apache#39341)

In previous versions, we used a method based on JDBC 4.2 to read data,
so it was equivalent to abandoning support for ojdbc6. However, we
recently found that a large number of users still use Oracle version
11g, which will have some unexpected compatibility issues when using
ojdbc8 to connect. Therefore, I use version verification to make it
compatible with both ojdbc6 and ojdbc8, so that good compatibility can
be obtained through ojdbc6, and better reading efficiency can be
obtained through ojdbc8.
yiguolei pushed a commit that referenced this pull request Aug 17, 2024
…ing version check (#39408)

pick (#39341)

In previous versions, we used a method based on JDBC 4.2 to read data,
so it was equivalent to abandoning support for ojdbc6. However, we
recently found that a large number of users still use Oracle version
11g, which will have some unexpected compatibility issues when using
ojdbc8 to connect. Therefore, I use version verification to make it
compatible with both ojdbc6 and ojdbc8, so that good compatibility can
be obtained through ojdbc6, and better reading efficiency can be
obtained through ojdbc8.
dataroaring pushed a commit that referenced this pull request Aug 17, 2024
…check (#39341)

In previous versions, we used a method based on JDBC 4.2 to read data,
so it was equivalent to abandoning support for ojdbc6. However, we
recently found that a large number of users still use Oracle version
11g, which will have some unexpected compatibility issues when using
ojdbc8 to connect. Therefore, I use version verification to make it
compatible with both ojdbc6 and ojdbc8, so that good compatibility can
be obtained through ojdbc6, and better reading efficiency can be
obtained through ojdbc8.
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.1.6-merged dev/3.0.2-merged doing reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants