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

[VL]Daily Update Velox Version (20231208) #3973

Merged
merged 1 commit into from
Dec 8, 2023
Merged

[VL]Daily Update Velox Version (20231208) #3973

merged 1 commit into from
Dec 8, 2023

Conversation

yma11
Copy link
Contributor

@yma11 yma11 commented Dec 7, 2023

What changes were proposed in this pull request?

[VL]Daily Update Velox Version (20231208) with following updates.

b908e7794 Add curl.cmake dependency for cpr (#7853)
af0d9ae93 Improve scheduled fuzzer workflow (#7842)
ddc347188 Add OderBy output stage spill (#7759)
61ab60ddc Ensure we only use subgroups if they are matched in url_extract_* presto functions (#7866)
db64f6fa8 Fix fuzz different output types for generated expression sets in expression fuzzer (#7910)
8374e669b Updating submodules
730dd5343 Extract toAggregateInfo(AggregationNode) helper function (#7859)
13d54c49d Set task wait timeout to 5 mins by default and docs fixes (#7911)
debe03c17 Fix function signature of approx_distinct(DECIMAL(a_precision,a_scale)) (#7905)
00c742213 Put back "[velox][PR] Add support for decimals in approx_distinct aggregate function"
e28d8d1b8 Fix race between task resume and task terminate (#7909)
f59e6df8a Introduce TypedExprs (#7907)
dffc39e8c Back out "Handle allocations that exceed PageRun limits" (#7908)
fb045b02e Fix PartitionedOutput with constant keys (#7906)
0c468476d Create ExpressionFuzzerVerifier::Options and move all flags to FuzzerRunner (#7634)
e963545e4 make skipFunctions, onlyFunctions, and specialForms as ExpressionFuzzer options. (#7882)
e7bebbd0d Fix merge conflict (#7902)
6a6d704f1 Updating submodules
af7d1ef4d Add str_to_map Spark function (#5842)
2e71d8e6a Fix HashStringAllocator::free for mixed multipart allocations (#7893)
4288f2b71 Allow UNKNOWN type in HivePartitionFunction (#7784)
25b75f5c9 Rename ByteStream as ByteOutputStream (#7867)

How was this patch tested?

CI/CD

Copy link

github-actions bot commented Dec 7, 2023

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

@yma11 yma11 changed the title [VL]Daily Update Velox Version (20231207) [VL]Daily Update Velox Version (20231208) Dec 8, 2023
Copy link

github-actions bot commented Dec 8, 2023

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Dec 8, 2023

Run Gluten Clickhouse CI

@yma11 yma11 merged commit a0b1330 into apache:main Dec 8, 2023
15 of 17 checks passed
@GlutenPerfBot
Copy link
Contributor

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query log/native_3973_time.csv log/native_master_12_07_2023_9c5314fd5_time.csv difference percentage
q1 35.97 33.97 -2.001 94.44%
q2 25.02 24.95 -0.073 99.71%
q3 36.89 37.91 1.023 102.77%
q4 35.81 37.39 1.587 104.43%
q5 71.02 71.30 0.288 100.41%
q6 7.15 5.38 -1.766 75.28%
q7 84.57 85.81 1.239 101.47%
q8 88.05 85.80 -2.257 97.44%
q9 123.45 123.51 0.058 100.05%
q10 46.26 45.87 -0.397 99.14%
q11 20.00 20.29 0.291 101.46%
q12 26.69 26.73 0.033 100.12%
q13 46.18 45.73 -0.444 99.04%
q14 18.59 16.69 -1.900 89.78%
q15 28.84 28.46 -0.381 98.68%
q16 15.58 15.56 -0.024 99.85%
q17 102.33 103.37 1.037 101.01%
q18 153.03 149.01 -4.015 97.38%
q19 14.30 12.84 -1.456 89.82%
q20 26.89 27.78 0.882 103.28%
q21 226.79 223.73 -3.057 98.65%
q22 13.16 13.39 0.230 101.75%
total 1246.57 1235.47 -11.104 99.11%

@yma11 yma11 deleted the 12-7 branch December 11, 2023 05:26
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.

3 participants