Skip to content

Conversation

@zhiqiang-dev
Copy link
Contributor

Change Logs

Hadoop, Hbase dependencies
Use HFIle code

Impact

No public API or user-facing feature change.

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

high

Documentation Update

Update compile command

Contributor's checklist

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

@zhiqiang-dev
Copy link
Contributor Author

zhiqiang-dev commented Oct 25, 2023

  • I would recommend that hadoop2 hadoop3 be maintained separately, otherwise the Hbase Module would need to be separated into different versions of Hadoop.

  • If I need to maintain both, I'll need to add the Profile configuration later and then separate Hbase into different Modules.

@zhiqiang-dev zhiqiang-dev force-pushed the master_hudi-support-hadoop3 branch from 07bc67e to 6a14f95 Compare October 25, 2023 08:51
@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

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Feb 26, 2024
Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

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

We're going to upgrade Hadoop dependency to Hadoop 3 in #11572 and deprecate HBase dependency soon (in fact, #10241 is merged to add a HFile reader independent of HBase). So IMO we should close this PR and prioritize those PRs.

@yihua
Copy link
Contributor

yihua commented Sep 11, 2024

Feel free to reopen the PR if the changes are still necessary based on the latest master.

@yihua yihua closed this Sep 11, 2024
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.

3 participants