Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[MINOR] Fix ArchivalUtils Logger name #10436

Merged
merged 1 commit into from
Jan 3, 2024
Merged

[MINOR] Fix ArchivalUtils Logger name #10436

merged 1 commit into from
Jan 3, 2024

Conversation

eric9204
Copy link
Contributor

@eric9204 eric9204 commented Jan 2, 2024

Change Logs

None

Impact

None

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

None

Documentation Update

None

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
    ticket number here and follow the instruction to make
    changes to the website.

Contributor's checklist

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

@hudi-bot
Copy link

hudi-bot commented Jan 2, 2024

CI report:

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

bvaradar
bvaradar approved these changes Jan 3, 2024
View reviewed changes
Copy link
Contributor

@bvaradar bvaradar left a comment

Choose a reason for hiding this comment

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

LGTM

@bvaradar bvaradar merged commit 1b74fc1 into apache:master Jan 3, 2024
31 checks passed
@eric9204 eric9204 deleted the fix_archivalUtils_logger_name branch January 4, 2024 01:47
yihua pushed a commit that referenced this pull request Feb 27, 2024
@eric9204 @yihua
[MINOR] Fix ArchivalUtils Logger named (#10436)
2601a0e 
Co-authored-by: dongsj <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bvaradar bvaradar bvaradar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eric9204 @hudi-bot @bvaradar