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

[GLUTEN-4831][VL] Support StructType in HashAggregate #4832

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

WangGuangxin
Copy link
Contributor

What changes were proposed in this pull request?

Support StructType in HashAggregate

(Fixes: #4831)

Copy link

github-actions bot commented Mar 1, 2024

#4831

Copy link

github-actions bot commented Mar 1, 2024

Run Gluten Clickhouse CI

2 similar comments
Copy link

github-actions bot commented Mar 1, 2024

Run Gluten Clickhouse CI

Copy link

github-actions bot commented Mar 1, 2024

Run Gluten Clickhouse CI

Copy link
Contributor

@PHILO-HE PHILO-HE left a comment

Choose a reason for hiding this comment

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

Please do a rebase and resolve conflicts. Thanks!

@@ -99,6 +99,7 @@ abstract class HashAggregateExecBaseTransformer(
true
case _: NumericType => true
case _: ArrayType => true
case s: StructType => true
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: remove s.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed and rebased. thanks.

@WangGuangxin WangGuangxin force-pushed the feat_groupby_structtype branch from 66e800c to 9bdb5a9 Compare March 7, 2024 00:03
Copy link

github-actions bot commented Mar 7, 2024

Run Gluten Clickhouse CI

Copy link
Contributor

@PHILO-HE PHILO-HE left a comment

Choose a reason for hiding this comment

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

Looks good! cc @zhztheplayer

@PHILO-HE PHILO-HE merged commit f9daf49 into apache:main Mar 7, 2024
20 checks passed
@GlutenPerfBot
Copy link
Contributor

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

query log/native_4832_time.csv log/native_master_03_06_2024_bddc3fd79_time.csv difference percentage
q1 35.48 38.77 3.290 109.27%
q2 24.11 24.36 0.251 101.04%
q3 40.59 39.60 -0.986 97.57%
q4 37.59 37.58 -0.007 99.98%
q5 71.30 69.92 -1.380 98.06%
q6 8.33 8.34 0.009 100.10%
q7 83.74 84.40 0.655 100.78%
q8 87.42 86.73 -0.683 99.22%
q9 126.48 119.49 -6.995 94.47%
q10 49.07 43.18 -5.885 88.01%
q11 20.37 20.86 0.496 102.44%
q12 30.85 28.05 -2.797 90.93%
q13 47.03 44.55 -2.487 94.71%
q14 18.98 17.03 -1.955 89.70%
q15 29.46 28.15 -1.315 95.54%
q16 14.59 14.07 -0.520 96.44%
q17 104.29 101.46 -2.835 97.28%
q18 144.10 145.26 1.166 100.81%
q19 14.75 13.99 -0.764 94.82%
q20 27.82 28.21 0.398 101.43%
q21 233.53 224.18 -9.347 96.00%
q22 15.05 14.99 -0.063 99.58%
total 1264.93 1233.17 -31.752 97.49%

taiyang-li pushed a commit to bigo-sg/gluten that referenced this pull request Mar 25, 2024
taiyang-li pushed a commit to bigo-sg/gluten that referenced this pull request Oct 8, 2024
taiyang-li pushed a commit to bigo-sg/gluten that referenced this pull request Oct 9, 2024
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.

[VL] Support StructType in HashAggregate
3 participants