Skip to content

Conversation

@TheR1sing3un
Copy link
Member

The previous judgment logic was a typo error.
Whether to push down depends on the parquetFilterPushDown, while whether to perform a record filter at the parquet level depends on the parquetRecordFilterEnabled

Describe the issue this Pull Request addresses

minor fix

Summary and Changelog

  1. push down pk filters to log file when spark enable parquetFilterPushDown

Impact

as above

Risk Level

low

Documentation Update

none

Contributor's checklist

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

…rPushDown`

1. push down pk filters to log file when spark enable `parquetFilterPushDown`

The previous judgment logic was a typo error.
Whether to push down depends on the `parquetFilterPushDown`,
while whether to perform a record filter at the parquet level depends on the `parquetRecordFilterEnabled`

Signed-off-by: TheR1sing3un <chaoyang@apache.org>
@github-actions github-actions bot added the size:XS PR with lines of changes in <= 10 label Nov 24, 2025
@hudi-bot
Copy link
Collaborator

CI report:

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

@yihua yihua added this to the release-1.2.0 milestone Nov 24, 2025
@yihua yihua self-assigned this Nov 25, 2025
@yihua yihua changed the title [MINOR] fix: push down pk filters to log file when spark enable parquetFilterPushDown fix: push down pk filters to log file when spark enable parquetFilterPushDown Nov 25, 2025
@danny0405 danny0405 merged commit 653f2ce into apache:master Nov 26, 2025
131 of 139 checks passed
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Dec 6, 2025
…rPushDown` (apache#14332)

1. push down pk filters to log file when spark enable `parquetFilterPushDown`

The previous judgment logic was a typo error.
Whether to push down depends on the `parquetFilterPushDown`,
while whether to perform a record filter at the parquet level depends on the `parquetRecordFilterEnabled`

Signed-off-by: TheR1sing3un <chaoyang@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS PR with lines of changes in <= 10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants