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 bug on query_partition_filter_required_schemas config property in Hive connector #22522

Merged

Conversation

okayhooni
Copy link
Contributor

@okayhooni okayhooni commented Jun 27, 2024

Description

  • fix minor bug on query-partition-filter-required-schemas option to be compatible with query-partition-filter-required
  • add minor test case for this fix
  • add missing description about that config on Hive connector docs, copied from v362 release note. (REF: https://trino.io/docs/current/release/release-362.html )

Additional context and related issues

  • related PR: Specify schema for which partition filter check can be applied #9106 by @Praveen2112
  • I need to declare query-partition-filter-required-schemas on catalog configuration, and switch on/off query-partition-filter-required option, depending on source/session by using session property manager
  • But, I found that query-partition-filter-required-schemas option is always applied even though query-partition-filter-required=false.
  • It is just a minor matter of boolean equation execution order of java

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Jun 27, 2024
@github-actions github-actions bot added docs hive Hive connector labels Jun 27, 2024
@okayhooni okayhooni force-pushed the bugfix/hive_partition_required_schemas branch 2 times, most recently from 733ad86 to 405c7c3 Compare June 28, 2024 11:25
@okayhooni
Copy link
Contributor Author

2 failing checks(trino-delta-lake & trino-iceberg) are not relevant to this one-line modification on hive connector. :)

@okayhooni okayhooni requested a review from ebyhr July 8, 2024 05:58
@okayhooni okayhooni force-pushed the bugfix/hive_partition_required_schemas branch 2 times, most recently from f9769f9 to b15418f Compare July 8, 2024 06:27
@ebyhr ebyhr changed the title chore) fix minor bug and add docs. for query-partition-filter-required-schemas config. of hive connector Fix bug on query_partition_filter_required_schemas config property in Hive connector Jul 8, 2024
@ebyhr ebyhr force-pushed the bugfix/hive_partition_required_schemas branch 2 times, most recently from 6255eee to b7a19c0 Compare July 8, 2024 23:43
@ebyhr ebyhr force-pushed the bugfix/hive_partition_required_schemas branch from b7a19c0 to 739e93a Compare July 8, 2024 23:50
@ebyhr ebyhr merged commit 7d1d686 into trinodb:master Jul 9, 2024
60 checks passed
@github-actions github-actions bot added this to the 452 milestone Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants