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](orc)fix orc reader missing column and filter missing column. #35583

Merged
merged 2 commits into from
May 31, 2024

Conversation

hubgeter
Copy link
Contributor

@hubgeter hubgeter commented May 29, 2024

Proposed changes

Fixed the problem that filtering with non-existent columns failed when reading orc tables

example :

mysql> select * from sc_test;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    5 |    6 | bac   |                 NULL | 20240226 |
|    3 |    4 | bac   |                 NULL | 20240227 |
|    1 |    2 | bac   |                    1 | 20240228 |
+------+------+-------+----------------------+----------+
3 rows in set (0.03 sec)

mysql> select * from sc_test where sensitive_query_flag is null;
0 rows in set (0.03 sec)

after :

mysql> select * from sc_test where sensitive_query_flag is null;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    3 |    4 | bac   |                 NULL | 20240227 |
|    5 |    6 | bac   |                 NULL | 20240226 |
+------+------+-------+----------------------+----------+
2 rows in set (0.03 sec)

Issue Number: close #xxx

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

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@hubgeter
Copy link
Contributor Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17609	4439	4287	4287
q2	2032	211	204	204
q3	10413	1258	1193	1193
q4	10205	854	832	832
q5	7480	2811	2725	2725
q6	230	136	138	136
q7	955	615	607	607
q8	9221	2163	2118	2118
q9	9379	6772	6801	6772
q10	9552	3949	3866	3866
q11	463	247	251	247
q12	457	286	219	219
q13	17558	3236	3191	3191
q14	271	222	221	221
q15	503	492	488	488
q16	536	397	397	397
q17	986	672	716	672
q18	8494	7953	7912	7912
q19	3542	1556	1547	1547
q20	668	318	320	318
q21	5202	4039	4078	4039
q22	360	288	286	286
Total cold run time: 116116 ms
Total hot run time: 42277 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4551	4408	4460	4408
q2	381	280	264	264
q3	3149	2967	2762	2762
q4	1907	1572	1641	1572
q5	5499	5514	5521	5514
q6	213	120	125	120
q7	2221	1859	1791	1791
q8	3295	3392	3396	3392
q9	8766	8784	8706	8706
q10	3908	3913	3821	3821
q11	603	500	510	500
q12	773	638	621	621
q13	16929	3185	3214	3185
q14	321	260	274	260
q15	545	481	496	481
q16	466	433	424	424
q17	1821	1497	1462	1462
q18	7745	7666	7448	7448
q19	1678	1587	1549	1549
q20	2001	1796	1747	1747
q21	10954	4671	4697	4671
q22	573	498	490	490
Total cold run time: 78299 ms
Total hot run time: 55188 ms
@doris-robot
Copy link

TPC-DS: Total hot run time: 172169 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 08c701f7613099dad8b604c5945bc264b753589c, data reload: false

query1	934	379	367	367
query2	6422	2360	2517	2360
query3	6634	206	211	206
query4	19157	17341	17427	17341
query5	4084	421	422	421
query6	244	154	166	154
query7	4591	300	292	292
query8	253	186	184	184
query9	8487	2469	2456	2456
query10	450	280	266	266
query11	10422	10158	10120	10120
query12	138	93	91	91
query13	1676	374	371	371
query14	9344	6844	7766	6844
query15	213	168	172	168
query16	7632	275	261	261
query17	1311	524	525	524
query18	1946	270	276	270
query19	198	149	153	149
query20	88	88	83	83
query21	197	134	127	127
query22	4253	3940	3901	3901
query23	33825	32941	33008	32941
query24	7119	2858	2834	2834
query25	594	349	360	349
query26	708	160	161	160
query27	2070	320	328	320
query28	4078	2113	2102	2102
query29	868	602	597	597
query30	246	151	153	151
query31	1012	778	760	760
query32	90	54	52	52
query33	486	277	275	275
query34	865	476	486	476
query35	730	661	603	603
query36	1037	952	905	905
query37	103	67	65	65
query38	2878	2766	2755	2755
query39	856	789	801	789
query40	192	125	125	125
query41	45	44	45	44
query42	99	96	95	95
query43	604	557	563	557
query44	1067	742	761	742
query45	175	168	167	167
query46	1054	771	719	719
query47	1854	1772	1750	1750
query48	369	316	296	296
query49	842	377	381	377
query50	771	392	390	390
query51	6682	6689	6635	6635
query52	112	87	91	87
query53	350	284	286	284
query54	544	435	442	435
query55	76	75	71	71
query56	302	239	248	239
query57	1138	1044	1043	1043
query58	238	208	208	208
query59	3584	3548	3144	3144
query60	281	259	253	253
query61	87	84	91	84
query62	614	460	462	460
query63	305	282	295	282
query64	8484	2238	1718	1718
query65	3194	3109	3160	3109
query66	802	320	321	320
query67	15128	14734	14886	14734
query68	4528	561	556	556
query69	443	268	273	268
query70	1209	1113	1147	1113
query71	372	276	260	260
query72	7315	5488	5329	5329
query73	724	330	327	327
query74	6129	5655	5595	5595
query75	3301	2655	2622	2622
query76	2401	1007	988	988
query77	419	274	274	274
query78	10267	9863	9699	9699
query79	2334	520	510	510
query80	1048	450	451	450
query81	547	226	227	226
query82	625	91	90	90
query83	255	177	172	172
query84	248	95	93	93
query85	1570	394	257	257
query86	493	323	327	323
query87	3275	3121	3109	3109
query88	4167	2430	2445	2430
query89	495	401	385	385
query90	2032	189	181	181
query91	123	97	96	96
query92	57	49	47	47
query93	2562	529	520	520
query94	1157	189	191	189
query95	409	313	310	310
query96	600	274	272	272
query97	3168	2973	2985	2973
query98	270	219	208	208
query99	1157	849	839	839
Total cold run time: 258932 ms
Total hot run time: 172169 ms
@doris-robot
Copy link

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

query1	0.04	0.04	0.04
query2	0.08	0.04	0.04
query3	0.23	0.06	0.06
query4	1.71	0.07	0.07
query5	0.49	0.48	0.60
query6	1.11	0.73	0.72
query7	0.02	0.02	0.01
query8	0.05	0.04	0.04
query9	0.56	0.49	0.49
query10	0.55	0.56	0.56
query11	0.16	0.11	0.11
query12	0.14	0.12	0.11
query13	0.60	0.59	0.59
query14	0.75	0.78	0.79
query15	0.84	0.80	0.80
query16	0.37	0.36	0.38
query17	0.99	1.04	1.03
query18	0.20	0.25	0.25
query19	1.75	1.70	1.70
query20	0.01	0.00	0.01
query21	15.44	0.73	0.68
query22	5.46	6.54	1.69
query23	18.31	1.44	1.27
query24	1.77	0.28	0.20
query25	0.14	0.09	0.08
query26	0.27	0.15	0.16
query27	0.09	0.08	0.08
query28	13.30	1.02	0.99
query29	12.89	3.33	3.28
query30	0.24	0.06	0.05
query31	2.88	0.37	0.38
query32	3.31	0.49	0.47
query33	2.81	2.89	2.93
query34	17.29	4.42	4.37
query35	4.54	4.50	4.56
query36	0.68	0.47	0.48
query37	0.17	0.18	0.16
query38	0.15	0.15	0.14
query39	0.04	0.04	0.03
query40	0.16	0.13	0.14
query41	0.09	0.05	0.04
query42	0.06	0.05	0.04
query43	0.04	0.03	0.03
Total cold run time: 110.78 s
Total hot run time: 30.35 s
@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 35.77% (9012/25197)
Line Coverage: 27.38% (74598/272496)
Region Coverage: 26.59% (38594/145162)
Branch Coverage: 23.47% (19682/83870)
Coverage Report: http://coverage.selectdb-in.cc/coverage/08c701f7613099dad8b604c5945bc264b753589c_08c701f7613099dad8b604c5945bc264b753589c/report/index.html

@hubgeter
Copy link
Contributor Author

run buildall

Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 36.29% (9232/25442)
Line Coverage: 27.63% (75703/273962)
Region Coverage: 26.85% (39203/145992)
Branch Coverage: 23.60% (19895/84284)
Coverage Report: http://coverage.selectdb-in.cc/coverage/d554f4880489cb28d273315b659161d00632d59b_d554f4880489cb28d273315b659161d00632d59b/report/index.html

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17635	4352	4264	4264
q2	2025	208	189	189
q3	10446	1258	1192	1192
q4	10190	793	753	753
q5	7461	2708	2722	2708
q6	223	134	137	134
q7	971	618	615	615
q8	9217	2110	2085	2085
q9	9567	6703	6809	6703
q10	9138	3911	4032	3911
q11	459	244	251	244
q12	506	225	227	225
q13	17444	3171	3328	3171
q14	254	217	211	211
q15	509	472	471	471
q16	488	388	417	388
q17	1005	700	750	700
q18	8603	7974	7793	7793
q19	6329	1607	1457	1457
q20	638	328	329	328
q21	5166	3290	3361	3290
q22	393	331	325	325
Total cold run time: 118667 ms
Total hot run time: 41157 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4572	4515	4485	4485
q2	374	270	274	270
q3	3146	2928	2978	2928
q4	1979	1626	1689	1626
q5	5385	5573	5671	5573
q6	219	129	132	129
q7	2334	1912	1978	1912
q8	3320	3483	3455	3455
q9	8760	8893	8933	8893
q10	4181	3791	3709	3709
q11	610	496	496	496
q12	805	657	649	649
q13	17654	3130	3171	3130
q14	279	279	280	279
q15	514	483	480	480
q16	505	434	436	434
q17	1803	1542	1494	1494
q18	8154	7397	7384	7384
q19	6435	1529	1590	1529
q20	2030	1742	1782	1742
q21	13646	4707	4836	4707
q22	584	507	547	507
Total cold run time: 87289 ms
Total hot run time: 55811 ms
AshinGau
AshinGau previously approved these changes May 31, 2024
@AshinGau
Copy link
Member

LGTM

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

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

Copy link
Contributor

PR approved by anyone and no changes requested.

morningman
morningman previously approved these changes May 31, 2024
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-H: Total hot run time: 43915 ms
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit d554f4880489cb28d273315b659161d00632d59b, data reload: false

------ Round 1 ----------------------------------
q1	5517	4582	4298	4298
q2	1625	197	193	193
q3	4304	3773	3707	3707
q4	10614	884	960	884
q5	8062	1994	2624	1994
q6	230	137	136	136
q7	20927	626	615	615
q8	9222	2199	2161	2161
q9	7047	6747	6604	6604
q10	7174	3902	3930	3902
q11	441	235	254	235
q12	522	221	240	221
q13	17379	3145	3207	3145
q14	268	229	232	229
q15	521	469	470	469
q16	523	417	404	404
q17	985	736	752	736
q18	8250	7875	7792	7792
q19	1737	1626	1559	1559
q20	655	332	328	328
q21	5181	4035	3973	3973
q22	382	330	340	330
Total cold run time: 111566 ms
Total hot run time: 43915 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4528	4444	4393	4393
q2	371	277	270	270
q3	3107	2845	2843	2843
q4	1954	1640	1729	1640
q5	5469	5525	5518	5518
q6	224	127	130	127
q7	2165	1814	1751	1751
q8	3207	3384	3379	3379
q9	8584	8612	8707	8612
q10	4042	3704	3652	3652
q11	566	487	486	486
q12	754	585	598	585
q13	12908	2979	2954	2954
q14	278	245	246	245
q15	517	464	485	464
q16	467	414	427	414
q17	1781	1468	1463	1463
q18	7665	7500	7498	7498
q19	7564	1659	1432	1432
q20	2000	1799	1759	1759
q21	4980	4706	4804	4706
q22	604	535	512	512
Total cold run time: 73735 ms
Total hot run time: 54703 ms
@doris-robot
Copy link

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

query1	913	374	368	368
query2	6449	2442	2380	2380
query3	6638	201	202	201
query4	19909	17379	17149	17149
query5	4127	407	404	404
query6	251	154	147	147
query7	4578	300	289	289
query8	336	293	283	283
query9	8462	2370	2346	2346
query10	455	280	260	260
query11	10532	10157	10064	10064
query12	158	94	88	88
query13	1650	378	397	378
query14	10013	7522	7578	7522
query15	250	187	188	187
query16	7152	269	261	261
query17	1227	550	539	539
query18	1927	274	268	268
query19	201	157	153	153
query20	87	88	83	83
query21	193	135	127	127
query22	4079	3928	3878	3878
query23	33402	33049	32988	32988
query24	5057	2838	2929	2838
query25	474	363	367	363
query26	694	156	156	156
query27	1844	311	317	311
query28	3745	2053	2043	2043
query29	862	594	594	594
query30	221	148	151	148
query31	918	756	748	748
query32	62	51	52	51
query33	455	258	272	258
query34	850	471	505	471
query35	701	601	613	601
query36	1006	936	915	915
query37	100	69	66	66
query38	2939	2792	2770	2770
query39	829	786	790	786
query40	197	125	123	123
query41	54	51	79	51
query42	100	94	92	92
query43	597	548	547	547
query44	1073	728	740	728
query45	187	175	166	166
query46	1063	753	720	720
query47	1873	1796	1778	1778
query48	359	292	299	292
query49	763	381	386	381
query50	770	391	385	385
query51	6795	6665	6698	6665
query52	103	88	91	88
query53	351	287	293	287
query54	528	426	432	426
query55	71	71	71	71
query56	264	239	243	239
query57	1110	1058	1040	1040
query58	218	238	204	204
query59	3488	3403	3394	3394
query60	277	256	262	256
query61	89	91	85	85
query62	554	465	437	437
query63	311	281	284	281
query64	2555	1837	1707	1707
query65	3165	3058	3125	3058
query66	751	325	338	325
query67	15243	14881	14632	14632
query68	4548	529	531	529
query69	440	267	263	263
query70	1171	1140	1093	1093
query71	378	269	264	264
query72	7461	2729	2514	2514
query73	705	323	319	319
query74	6093	5625	5606	5606
query75	3300	2653	2589	2589
query76	2196	978	1050	978
query77	381	259	261	259
query78	10303	9740	9777	9740
query79	1066	520	499	499
query80	854	423	430	423
query81	509	217	219	217
query82	458	90	88	88
query83	245	191	168	168
query84	245	84	83	83
query85	858	270	264	264
query86	401	289	312	289
query87	3324	3136	3131	3131
query88	3518	2360	2354	2354
query89	466	406	382	382
query90	2074	192	184	184
query91	132	110	107	107
query92	58	52	49	49
query93	1262	500	495	495
query94	1117	197	254	197
query95	404	308	309	308
query96	575	275	265	265
query97	3256	2993	2984	2984
query98	234	216	220	216
query99	1182	856	840	840
Total cold run time: 245869 ms
Total hot run time: 169637 ms
@doris-robot
Copy link

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

query1	0.03	0.03	0.03
query2	0.08	0.04	0.04
query3	0.24	0.05	0.05
query4	1.68	0.08	0.10
query5	0.49	0.49	0.49
query6	1.13	0.76	0.73
query7	0.02	0.01	0.01
query8	0.06	0.04	0.04
query9	0.54	0.49	0.48
query10	0.55	0.55	0.54
query11	0.16	0.11	0.12
query12	0.14	0.12	0.12
query13	0.59	0.59	0.58
query14	0.77	0.77	0.77
query15	0.82	0.81	0.81
query16	0.37	0.36	0.35
query17	0.98	1.03	1.02
query18	0.20	0.24	0.25
query19	1.85	1.65	1.65
query20	0.01	0.01	0.02
query21	15.54	0.66	0.65
query22	3.90	7.29	1.85
query23	18.27	1.35	1.27
query24	1.93	0.23	0.21
query25	0.15	0.08	0.08
query26	0.27	0.16	0.17
query27	0.07	0.08	0.09
query28	13.27	1.01	1.00
query29	13.24	3.41	3.38
query30	0.24	0.06	0.05
query31	2.86	0.38	0.38
query32	3.30	0.47	0.46
query33	2.87	2.91	2.92
query34	17.16	4.45	4.49
query35	4.48	4.49	4.62
query36	0.68	0.46	0.49
query37	0.17	0.15	0.16
query38	0.16	0.14	0.14
query39	0.05	0.03	0.04
query40	0.17	0.14	0.14
query41	0.09	0.04	0.05
query42	0.06	0.05	0.05
query43	0.04	0.03	0.04
Total cold run time: 109.68 s
Total hot run time: 30.61 s
@hubgeter hubgeter dismissed stale reviews from morningman and AshinGau via b2ac469 May 31, 2024 11:33
@hubgeter
Copy link
Contributor Author

run buildall

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label May 31, 2024
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17618	4269	4210	4210
q2	2022	194	197	194
q3	11211	1210	1150	1150
q4	10225	824	745	745
q5	7570	2700	2681	2681
q6	220	133	137	133
q7	949	625	601	601
q8	9291	2081	2059	2059
q9	9310	6597	6595	6595
q10	9424	3713	3689	3689
q11	469	248	243	243
q12	500	222	223	222
q13	17773	3004	2978	2978
q14	263	230	228	228
q15	519	473	477	473
q16	531	391	378	378
q17	959	695	684	684
q18	8156	7378	7421	7378
q19	2220	1557	1499	1499
q20	629	306	305	305
q21	4944	3281	3220	3220
q22	373	328	337	328
Total cold run time: 115176 ms
Total hot run time: 39993 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4367	4238	4255	4238
q2	368	268	278	268
q3	2940	2777	2686	2686
q4	1864	1573	1585	1573
q5	5208	5259	5214	5214
q6	214	123	125	123
q7	2149	1712	1750	1712
q8	3205	3276	3275	3275
q9	8263	8272	8328	8272
q10	3887	3692	3670	3670
q11	583	492	501	492
q12	772	576	598	576
q13	16681	2951	2992	2951
q14	293	245	274	245
q15	509	474	473	473
q16	469	408	412	408
q17	1744	1468	1464	1464
q18	7555	7452	7570	7452
q19	2224	1513	1405	1405
q20	1967	1754	1811	1754
q21	4788	4675	4751	4675
q22	606	573	529	529
Total cold run time: 70656 ms
Total hot run time: 53455 ms
@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 36.29% (9232/25442)
Line Coverage: 27.63% (75707/273963)
Region Coverage: 26.85% (39194/145992)
Branch Coverage: 23.60% (19892/84284)
Coverage Report: http://coverage.selectdb-in.cc/coverage/b2ac469af0da38332365a623e23c5b18cd590fd7_b2ac469af0da38332365a623e23c5b18cd590fd7/report/index.html

@doris-robot
Copy link

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

query1	903	369	371	369
query2	6449	2506	2402	2402
query3	6649	207	205	205
query4	19335	17421	16930	16930
query5	4126	418	435	418
query6	254	162	154	154
query7	4578	306	290	290
query8	347	293	295	293
query9	8586	2415	2457	2415
query10	447	292	250	250
query11	10588	9991	9914	9914
query12	132	92	84	84
query13	1632	361	358	358
query14	10065	7486	6924	6924
query15	243	189	188	188
query16	7827	260	256	256
query17	1863	539	510	510
query18	1964	281	294	281
query19	194	162	153	153
query20	91	83	92	83
query21	197	130	125	125
query22	4308	3944	3793	3793
query23	33512	32928	32976	32928
query24	12044	2825	2760	2760
query25	683	351	361	351
query26	1835	157	154	154
query27	3039	321	312	312
query28	7492	2073	2059	2059
query29	1168	610	595	595
query30	291	147	151	147
query31	960	743	731	731
query32	97	52	54	52
query33	765	267	263	263
query34	1089	457	484	457
query35	723	621	576	576
query36	1057	952	923	923
query37	264	67	69	67
query38	2888	2723	2753	2723
query39	852	800	780	780
query40	281	126	130	126
query41	54	52	52	52
query42	102	100	94	94
query43	601	552	563	552
query44	1216	744	754	744
query45	203	169	166	166
query46	1064	699	722	699
query47	1830	1728	1795	1728
query48	385	316	305	305
query49	1192	387	372	372
query50	754	399	378	378
query51	6738	6502	6614	6502
query52	114	92	90	90
query53	356	288	285	285
query54	1000	428	419	419
query55	73	73	73	73
query56	256	250	247	247
query57	1117	1036	1050	1036
query58	258	236	205	205
query59	3508	3130	3228	3130
query60	275	266	264	264
query61	96	88	91	88
query62	660	449	437	437
query63	307	297	282	282
query64	9875	2326	1816	1816
query65	3661	3118	3169	3118
query66	1397	333	342	333
query67	15299	15224	14778	14778
query68	4512	539	537	537
query69	447	275	278	275
query70	1205	1115	1123	1115
query71	418	277	284	277
query72	7678	2873	2686	2686
query73	707	322	320	320
query74	5891	5595	5591	5591
query75	3487	2617	2626	2617
query76	2941	1027	983	983
query77	404	275	280	275
query78	10369	9833	9817	9817
query79	2719	523	525	523
query80	2054	459	458	458
query81	539	230	221	221
query82	872	93	94	93
query83	290	187	195	187
query84	284	91	92	91
query85	2075	335	333	333
query86	477	297	305	297
query87	3319	3104	3124	3104
query88	4487	2389	2363	2363
query89	478	386	372	372
query90	2032	190	182	182
query91	124	97	97	97
query92	60	47	54	47
query93	4670	522	493	493
query94	1173	193	182	182
query95	387	306	308	306
query96	602	273	268	268
query97	3186	3029	3012	3012
query98	247	222	218	218
query99	1208	828	836	828
Total cold run time: 281551 ms
Total hot run time: 168760 ms
@doris-robot
Copy link

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

query1	0.04	0.03	0.03
query2	0.09	0.05	0.04
query3	0.22	0.05	0.05
query4	1.68	0.08	0.09
query5	0.50	0.48	0.49
query6	1.12	0.72	0.71
query7	0.02	0.01	0.01
query8	0.04	0.04	0.04
query9	0.53	0.47	0.47
query10	0.52	0.55	0.53
query11	0.15	0.11	0.12
query12	0.15	0.12	0.12
query13	0.59	0.59	0.61
query14	0.77	0.77	0.78
query15	0.83	0.82	0.82
query16	0.35	0.35	0.36
query17	0.99	0.95	0.98
query18	0.22	0.26	0.25
query19	1.75	1.66	1.70
query20	0.02	0.01	0.01
query21	15.69	0.66	0.65
query22	4.19	6.96	2.37
query23	18.32	1.40	1.25
query24	2.10	0.21	0.20
query25	0.15	0.09	0.08
query26	0.24	0.16	0.16
query27	0.08	0.08	0.08
query28	13.27	1.02	0.99
query29	13.29	3.26	3.27
query30	0.24	0.06	0.05
query31	2.88	0.38	0.38
query32	3.30	0.48	0.46
query33	2.91	2.88	2.89
query34	17.23	4.38	4.43
query35	4.48	4.47	4.48
query36	0.64	0.46	0.48
query37	0.18	0.15	0.16
query38	0.15	0.14	0.15
query39	0.05	0.03	0.03
query40	0.17	0.15	0.15
query41	0.09	0.05	0.04
query42	0.05	0.05	0.05
query43	0.04	0.04	0.04
Total cold run time: 110.32 s
Total hot run time: 30.79 s
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

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

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

yiguolei pushed a commit that referenced this pull request May 31, 2024
## Proposed changes
bp #35583 
Issue Number: close #xxx

<!--Describe your changes.-->
morningman pushed a commit that referenced this pull request May 31, 2024
@morningman morningman merged commit dd9beff into apache:master May 31, 2024
28 of 33 checks passed
@morningman morningman mentioned this pull request Jun 1, 2024
dataroaring pushed a commit that referenced this pull request Jun 4, 2024
…35583)

Fixed the problem that filtering with non-existent columns failed when
reading orc tables

example : 
```
mysql> select * from sc_test;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    5 |    6 | bac   |                 NULL | 20240226 |
|    3 |    4 | bac   |                 NULL | 20240227 |
|    1 |    2 | bac   |                    1 | 20240228 |
+------+------+-------+----------------------+----------+
3 rows in set (0.03 sec)

mysql> select * from sc_test where sensitive_query_flag is null;
0 rows in set (0.03 sec)
```
after : 
```
mysql> select * from sc_test where sensitive_query_flag is null;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    3 |    4 | bac   |                 NULL | 20240227 |
|    5 |    6 | bac   |                 NULL | 20240226 |
+------+------+-------+----------------------+----------+
2 rows in set (0.03 sec)

```
seawinde pushed a commit to seawinde/doris that referenced this pull request Jun 5, 2024
…pache#35583)

Fixed the problem that filtering with non-existent columns failed when
reading orc tables

example : 
```
mysql> select * from sc_test;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    5 |    6 | bac   |                 NULL | 20240226 |
|    3 |    4 | bac   |                 NULL | 20240227 |
|    1 |    2 | bac   |                    1 | 20240228 |
+------+------+-------+----------------------+----------+
3 rows in set (0.03 sec)

mysql> select * from sc_test where sensitive_query_flag is null;
0 rows in set (0.03 sec)
```
after : 
```
mysql> select * from sc_test where sensitive_query_flag is null;
+------+------+-------+----------------------+----------+
| id   | type | state | sensitive_query_flag | dt       |
+------+------+-------+----------------------+----------+
|    3 |    4 | bac   |                 NULL | 20240227 |
|    5 |    6 | bac   |                 NULL | 20240226 |
+------+------+-------+----------------------+----------+
2 rows in set (0.03 sec)

```
AshinGau added a commit that referenced this pull request Jun 19, 2024
…olumn (#36189)

## Proposed changes

follow #35583, fix parquet reader.
dataroaring pushed a commit that referenced this pull request Jun 21, 2024
…olumn (#36189)

## Proposed changes

follow #35583, fix parquet reader.
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.11-merged dev/2.1.4-merged reviewed
5 participants