-
Notifications
You must be signed in to change notification settings - Fork 390
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-3559][VL] Fix unit tests in GlutenParquetV1FilterSuite and GlutenParquetV2FilterSuite #4381
Conversation
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
@JkSelf kindly review |
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
@JkSelf @zhli1142015 Kindly review |
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.
LGTM. Thanks.
===== Performance report for TPCH SF2000 with Velox backend, for reference only ====
|
What changes were proposed in this pull request?
This PR enables following tests in the suite GlutenParquetV1FilterSuite and GlutenParquetV2FilterSuite: "filter pushdown - StringStartsWith"
Please note that #3559 has mentioned test 'filter pushdown - StringPredicate' but that's similar to "filter pushdown - StringStartsWith" of Spark33 and so will remain disabled (as similar test is disabled in Spark33).
(Fixes: #3559)
How was this patch tested?
Ran this UT to confirm the fix