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

Fix checkstyle failure in BookieRackAffinityMapping #13780

Closed
wants to merge 1 commit into from
Closed

Fix checkstyle failure in BookieRackAffinityMapping #13780

wants to merge 1 commit into from

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jan 17, 2022

Motivation

Modifications

  • replace Guava usage with Commons Lang

@lhotari
Fix checkstyle failure in BookieRackAffinityMapping
370a4a0 
- replace Guava with Commons Lang
@lhotari lhotari added the doc-not-needed Your PR changes do not impact docs label Jan 17, 2022
@lhotari lhotari self-assigned this Jan 17, 2022
@gaozhangmin
Copy link
Contributor

#13772 already fixed it. @lhotari

@lhotari
Copy link
Member Author

lhotari commented Jan 17, 2022

#13772 already fixed it. @lhotari

thank you @gaozhangmin . I merged that one now.

@lhotari lhotari closed this Jan 17, 2022
@lhotari lhotari deleted the lh-fix-checkstyle-in-BookieRackAffinityMapping branch January 17, 2022 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hangc0276 hangc0276 Awaiting requested review from hangc0276

@eolivelli eolivelli Awaiting requested review from eolivelli

Assignees

@lhotari lhotari

Labels
doc-not-needed Your PR changes do not impact docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lhotari @gaozhangmin