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

Fix partition predicate pushdown containing null in Glue #13122

Merged
merged 6 commits into from
Jul 20, 2022

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Jul 8, 2022

Description

Fix partition predicate pushdown containing null in Glue
Fixes #13083

Documentation

(x) No documentation is needed.

Release notes

(x) Release notes entries required with the following suggested text:

# Hive, Iceberg, Delta Lake
* Fix incorrect results when using the Glue metastore and queries contain `IS NULL` with additional filters. ({issue}`13122 `)

@ebyhr ebyhr force-pushed the ebi/hive-glue-partition-issue branch from 2a75a34 to b43f67f Compare July 19, 2022 04:58
@ebyhr ebyhr marked this pull request as ready for review July 19, 2022 06:23
@findepi
Copy link
Member

findepi commented Jul 19, 2022

42 successful, 1 failing, and 1 in progress checks
ci / test (plugin/trino-delta-lake) (pull_request) Failing after 18m — test (plugin/trino-delta-lake)
ci / pt (default, suite-2, ) (pull_request) In progress — This check has started...

@ebyhr ebyhr force-pushed the ebi/hive-glue-partition-issue branch from f4b28c7 to f1f80e6 Compare July 20, 2022 04:32
@ebyhr ebyhr merged commit 767e6bb into master Jul 20, 2022
@ebyhr ebyhr deleted the ebi/hive-glue-partition-issue branch July 20, 2022 06:31
@ebyhr ebyhr mentioned this pull request Jul 20, 2022
@github-actions github-actions bot added this to the 391 milestone Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Predicate pushdown with NULL on Glue return incorrect result
3 participants