Skip to content

fix violation checkstyle - #13778

Closed
gaozhangmin wants to merge 1 commit into
apache:masterfrom
gaozhangmin:violation-checkstyle-Strings
Closed

fix violation checkstyle#13778
gaozhangmin wants to merge 1 commit into
apache:masterfrom
gaozhangmin:violation-checkstyle-Strings

Conversation

@gaozhangmin

@gaozhangmin gaozhangmin commented Jan 17, 2022

Copy link
Copy Markdown
Contributor

Fixes #13683 check style violation:

Error:  src/main/java/org/apache/pulsar/bookie/rackawareness/BookieRackAffinityMapping.java:[21] (regexp) 

RegexpSinglelineJava: Line matches the illegal pattern 'com\.google\.api\.client\.util\.(ByteStreams|Charsets|Collections2|Joiner|Lists|Maps|Objects|Preconditions|Sets|Strings|Throwables)'.

Documentation

Check the box below or label this PR directly (if you have committer privilege).

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@github-actions

Copy link
Copy Markdown

@gaozhangmin:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@gaozhangmin

Copy link
Copy Markdown
Contributor Author

@hangc0276 PTAL

@github-actions

Copy link
Copy Markdown

@gaozhangmin:Thanks for providing doc info!

@github-actions github-actions Bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Jan 17, 2022
@gaozhangmin

Copy link
Copy Markdown
Contributor Author

#13772 already fixed it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants