Skip to content

[HUDI-7589] Add API to create HoodieStorage in HoodieIOFactory#11206

Closed
jonvex wants to merge 12 commits intoapache:masterfrom
jonvex:add_hoodie_storage_to_io_factory
Closed

[HUDI-7589] Add API to create HoodieStorage in HoodieIOFactory#11206
jonvex wants to merge 12 commits intoapache:masterfrom
jonvex:add_hoodie_storage_to_io_factory

Conversation

@jonvex
Copy link
Contributor

@jonvex jonvex commented May 13, 2024

Change Logs

  1. Cache the StorageConfig in HoodieIOFactory so it doesn't need to be passed as a param multiple times for the same reader
  2. Add getStorage method to HoodieIOFactory

Impact

Get rid of most reflection calls to HoodieStorageUtils

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:XL PR with lines of changes > 1000 label May 13, 2024
@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

@jonvex jonvex closed this May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL PR with lines of changes > 1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants