Skip to content

Conversation

ad1happy2go
Copy link
Collaborator

@ad1happy2go ad1happy2go commented Oct 7, 2025

Describe the issue this Pull Request addresses

Fixes #14058

While running hudi on databricks, the code failed as InterpretedPredicate was expecting two parameters in the constructor

Summary and Changelog

We can use Hudi with Databricks Runtime seemlessly. Before we were facing issues while adding partition predicate to hudi reads with Databricks runtime

Impact

none

Risk Level

none

Documentation Update

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@ad1happy2go ad1happy2go requested a review from yihua October 7, 2025 11:00
@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Oct 7, 2025
@ad1happy2go ad1happy2go force-pushed the databricks_partition_predicate_fix branch from 2dcbc47 to 43b5324 Compare October 7, 2025 11:56
@ad1happy2go ad1happy2go changed the title feat: Databricks partition predicate fix feat: Partition predicate fix for Databricks runtime support Oct 7, 2025
@hudi-bot
Copy link

hudi-bot commented Oct 7, 2025

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

Copy link
Contributor

@nsivabalan nsivabalan left a comment

Choose a reason for hiding this comment

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

have we tested this?
and I assume it may not be easy to write any functional tests on this to go w/ CI?

@nsivabalan nsivabalan self-assigned this Oct 8, 2025
@ad1happy2go
Copy link
Collaborator Author

have we tested this? and I assume it may not be easy to write any functional tests on this to go w/ CI?

Yes @nsivabalan I have tested this and reads are working fine in databricks also now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix PartitonPredicate constructor issue with Databricks Runtime version

3 participants