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.

Sign up for GitHub

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

Jump to bottom

[fix]fixed complie error when build be #48410

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

[fix]fixed complie error when build be #48410

wants to merge 1 commit into from

Conversation

cjj2010
Copy link
Contributor

@cjj2010 cjj2010 commented Feb 27, 2025 •
edited
Loading

What problem does this PR solve?

Issue Number: close #48403

Problem Summary:
Add judgment on bshuf_compressibility lz4-bound lib

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 complie be error only build base mac pro of inter x64

  • 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

@Thearas
Copy link
Contributor

Thearas commented Feb 27, 2025

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?

@cjj2010
Copy link
Contributor Author

cjj2010 commented Feb 27, 2025

run buildall

@doris-robot
Copy link

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

------ Round 1 ----------------------------------
q1	17602	5292	5157	5157
q2	2061	293	178	178
q3	10402	1279	784	784
q4	10212	1047	568	568
q5	7524	2382	2366	2366
q6	195	167	134	134
q7	929	767	621	621
q8	9312	1287	1155	1155
q9	4923	4820	4702	4702
q10	6845	2320	1898	1898
q11	521	280	270	270
q12	357	360	226	226
q13	17787	3753	3124	3124
q14	241	232	212	212
q15	511	484	461	461
q16	641	619	578	578
q17	591	870	350	350
q18	6898	6204	6179	6179
q19	1751	952	552	552
q20	329	327	196	196
q21	2932	2333	2101	2101
q22	366	337	303	303
Total cold run time: 102930 ms
Total hot run time: 32115 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5296	5197	5192	5192
q2	238	338	241	241
q3	2182	2693	2339	2339
q4	1437	1814	1371	1371
q5	4223	4141	4158	4141
q6	210	165	124	124
q7	1889	1871	1756	1756
q8	2629	2695	2583	2583
q9	7264	7170	7183	7170
q10	2970	3282	2780	2780
q11	597	525	492	492
q12	726	796	638	638
q13	3393	4005	3285	3285
q14	280	300	264	264
q15	488	474	470	470
q16	646	687	665	665
q17	1186	1657	1328	1328
q18	7631	7509	7382	7382
q19	835	866	868	866
q20	1999	2001	1887	1887
q21	5522	5082	4679	4679
q22	633	572	563	563
Total cold run time: 52274 ms
Total hot run time: 50216 ms

@doris-robot
Copy link

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

query1	1318	955	960	955
query2	6242	1891	1874	1874
query3	10967	4501	4608	4501
query4	54687	25877	23505	23505
query5	4837	668	501	501
query6	309	191	183	183
query7	4885	524	286	286
query8	305	253	233	233
query9	5515	2602	2606	2602
query10	431	332	261	261
query11	15199	15158	14906	14906
query12	167	113	105	105
query13	1062	526	401	401
query14	11119	6997	7074	6997
query15	211	209	197	197
query16	7097	698	492	492
query17	1092	752	591	591
query18	1600	426	327	327
query19	218	223	172	172
query20	134	129	124	124
query21	208	132	111	111
query22	4496	4300	4317	4300
query23	34165	33425	33139	33139
query24	5799	2479	2512	2479
query25	469	481	411	411
query26	718	284	160	160
query27	1764	516	334	334
query28	2770	2461	2475	2461
query29	579	583	442	442
query30	221	186	158	158
query31	893	904	850	850
query32	81	67	60	60
query33	509	383	333	333
query34	794	895	520	520
query35	828	887	765	765
query36	945	1011	934	934
query37	116	99	78	78
query38	4331	4238	4249	4238
query39	1493	1472	1445	1445
query40	210	125	107	107
query41	57	61	53	53
query42	124	108	108	108
query43	534	542	470	470
query44	1358	829	812	812
query45	187	190	169	169
query46	930	1105	683	683
query47	1881	1874	1803	1803
query48	397	427	309	309
query49	730	516	439	439
query50	805	808	438	438
query51	4317	4333	4356	4333
query52	114	109	105	105
query53	232	284	192	192
query54	489	502	445	445
query55	88	90	83	83
query56	276	299	263	263
query57	1165	1156	1124	1124
query58	243	248	255	248
query59	2753	2868	2689	2689
query60	285	304	276	276
query61	124	121	121	121
query62	768	744	697	697
query63	243	198	202	198
query64	2156	1084	732	732
query65	3367	3225	3264	3225
query66	784	395	320	320
query67	15664	15389	15264	15264
query68	6754	889	495	495
query69	544	315	274	274
query70	1308	1106	1101	1101
query71	505	295	268	268
query72	5745	3690	3879	3690
query73	1523	776	348	348
query74	8988	9191	8973	8973
query75	4018	3221	2724	2724
query76	4216	1201	791	791
query77	713	382	280	280
query78	10076	10106	9320	9320
query79	2888	848	584	584
query80	725	599	449	449
query81	507	278	237	237
query82	665	131	93	93
query83	231	172	150	150
query84	295	94	77	77
query85	772	350	308	308
query86	361	311	290	290
query87	4418	4445	4584	4445
query88	3727	2228	2291	2228
query89	420	330	288	288
query90	1896	201	196	196
query91	130	134	108	108
query92	78	68	56	56
query93	2002	1049	565	565
query94	652	413	291	291
query95	351	272	258	258
query96	501	580	273	273
query97	3335	3415	3313	3313
query98	237	207	198	198
query99	1451	1393	1301	1301
Total cold run time: 298887 ms
Total hot run time: 192109 ms

@doris-robot
Copy link

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

query1	0.03	0.04	0.04
query2	0.08	0.03	0.04
query3	0.23	0.07	0.07
query4	1.62	0.10	0.10
query5	0.54	0.54	0.55
query6	1.19	0.73	0.72
query7	0.03	0.02	0.02
query8	0.04	0.03	0.03
query9	0.59	0.53	0.52
query10	0.57	0.56	0.57
query11	0.15	0.11	0.11
query12	0.14	0.12	0.11
query13	0.61	0.59	0.60
query14	2.81	2.71	2.82
query15	0.92	0.84	0.85
query16	0.38	0.36	0.39
query17	1.02	1.02	1.00
query18	0.21	0.20	0.20
query19	1.90	1.84	1.90
query20	0.01	0.01	0.01
query21	15.35	0.86	0.53
query22	0.74	1.13	0.66
query23	14.98	1.39	0.62
query24	6.88	1.90	1.07
query25	0.55	0.20	0.22
query26	0.63	0.15	0.13
query27	0.05	0.04	0.04
query28	9.74	0.86	0.41
query29	12.53	3.90	3.29
query30	0.26	0.09	0.06
query31	2.81	0.58	0.39
query32	3.24	0.55	0.47
query33	3.02	2.98	3.03
query34	15.76	5.06	4.48
query35	4.50	4.53	4.47
query36	0.65	0.50	0.51
query37	0.09	0.06	0.06
query38	0.04	0.04	0.03
query39	0.03	0.02	0.02
query40	0.17	0.13	0.13
query41	0.08	0.02	0.02
query42	0.03	0.02	0.02
query43	0.03	0.03	0.03
Total cold run time: 105.23 s
Total hot run time: 31.01 s

@doris-robot
Copy link

BE UT Coverage Report

Increment line coverage 100.00% (1/1) 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 44.80% (11943/26656)
Line Coverage 34.23% (99925/291934)
Region Coverage 33.38% (51133/153201)
Branch Coverage 28.93% (25682/88766)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Enhancement] Support build be for mac pro base Intel Xeon W
3 participants
@cjj2010 @Thearas @doris-robot