-
Notifications
You must be signed in to change notification settings - Fork 447
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] Spark assert_true function support #6329
Conversation
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
4920289
to
e667e65
Compare
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
de1c00b
to
0e286d7
Compare
Run Gluten Clickhouse CI |
0e286d7
to
247878c
Compare
Run Gluten Clickhouse CI |
@gaoyangxiaozhu Would you like to rebase ? Thanks. |
…gayangya/assert_true
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! One minor comment.
backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxSparkPlanExecApi.scala
Outdated
Show resolved
Hide resolved
Run Gluten Clickhouse CI |
…gayangya/assert_true
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
@PHILO-HE help sign and merge, thanks! |
This reverts commit 6f189c7.
This reverts commit 6f189c7.
Depends on facebookincubator/velox#10110