Skip to content

[HUDI-9186] Remove tableState from HoodieFileGroupReaderBasedParquetFileFormat constructor #12981

Merged
danny0405 merged 2 commits intoapache:masterfrom
jonvex:hoodie_table_state_remove_fg_reader
Mar 16, 2025
Merged

[HUDI-9186] Remove tableState from HoodieFileGroupReaderBasedParquetFileFormat constructor #12981
danny0405 merged 2 commits intoapache:masterfrom
jonvex:hoodie_table_state_remove_fg_reader

Conversation

@jonvex
Copy link
Contributor

@jonvex jonvex commented Mar 14, 2025

Change Logs

tableState has fields that we are not using

Impact

don't calculate unnecessary fields

Risk level (write none, low medium or high below)

low

Documentation Update

N/A

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Mar 14, 2025
@jonvex jonvex requested a review from yihua March 14, 2025 23:04
@yihua yihua self-assigned this Mar 15, 2025
@apache apache deleted a comment from hudi-bot Mar 15, 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

@danny0405 danny0405 merged commit 54dcbbb into apache:master Mar 16, 2025
43 checks passed
voonhous pushed a commit to voonhous/hudi that referenced this pull request Apr 8, 2025
…ileFormat constructor (apache#12981)

* don't use tablestate for filegroup reader
* revert change for multible base format

---------

Co-authored-by: Jonathan Vexler <=>
(cherry picked from commit 54dcbbb)
voonhous pushed a commit to voonhous/hudi that referenced this pull request Apr 9, 2025
…ileFormat constructor (apache#12981)

* don't use tablestate for filegroup reader
* revert change for multible base format

---------

Co-authored-by: Jonathan Vexler <=>
(cherry picked from commit 54dcbbb)
voonhous pushed a commit to voonhous/hudi that referenced this pull request Apr 15, 2025
…ileFormat constructor (apache#12981)

* don't use tablestate for filegroup reader
* revert change for multible base format

---------

Co-authored-by: Jonathan Vexler <=>
(cherry picked from commit 54dcbbb)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants