Skip to content

Conversation

@xushiyan
Copy link
Member

@xushiyan xushiyan commented Dec 3, 2022

Change Logs

Relocate hbase dependencies in root pom.xml for reuse.

Impact

NA

Risk level

Low

Documentation Update

NA

Contributor's checklist

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

@xushiyan xushiyan requested a review from yihua December 3, 2022 06:31
@xushiyan xushiyan added dependencies Dependency updates priority:high Significant impact; potential bugs labels Dec 3, 2022
@xushiyan xushiyan force-pushed the HUDI-3673-fix-hbase-dep branch from 2f69501 to f3d658b Compare December 29, 2022 16:24
Copy link
Member

@codope codope left a comment

Choose a reason for hiding this comment

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

LGTM. Validated presto bundle manually.

@codope codope force-pushed the HUDI-3673-fix-hbase-dep branch from f3d658b to 6ff17db Compare January 23, 2023 05:58
@codope codope force-pushed the HUDI-3673-fix-hbase-dep branch from 6ff17db to b63b20e Compare January 24, 2023 05:11
@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

@codope
Copy link
Member

codope commented Jan 24, 2023

Merging it as flaky testUpsertsContinuousModeWithMultipleWritersForConflicts is unrelated to the change and that should be fixed by #7720

@codope codope merged commit d439fab into apache:master Jan 24, 2023
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Jan 31, 2023
danny0405 pushed a commit to danny0405/hudi that referenced this pull request Mar 23, 2023
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Mar 23, 2023
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency updates priority:high Significant impact; potential bugs

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants