Skip to content

MINOR: Remove warning in generating RoundRobinAssignor javadoc#7073

Merged
omkreddy merged 1 commit into
apache:trunkfrom
dongjinleekr:cleanup/201907
Nov 30, 2019
Merged

MINOR: Remove warning in generating RoundRobinAssignor javadoc#7073
omkreddy merged 1 commit into
apache:trunkfrom
dongjinleekr:cleanup/201907

Conversation

@dongjinleekr

@dongjinleekr dongjinleekr commented Jul 11, 2019

Copy link
Copy Markdown
Contributor 
> Task :clients:javadoc
clients/src/main/java/org/apache/kafka/clients/consumer/RoundRobinAssignor.java:78: warning - invalid usage of tag >
clients/src/main/java/org/apache/kafka/clients/consumer/RoundRobinAssignor.java:78: warning - invalid usage of tag >
clients/src/main/java/org/apache/kafka/clients/consumer/RoundRobinAssignor.java:78: warning - invalid usage of tag >
3 warnings

'>' character in javadoc should be replaced with >.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@dongjinleekr

dongjinleekr commented Oct 10, 2019

Copy link
Copy Markdown
Contributor Author

Rebased onto the latest trunk.

@dongjinleekr

dongjinleekr commented Nov 25, 2019

Copy link
Copy Markdown
Contributor Author

Rebased onto the latest trunk.

cc/ @hachikuji @bbejeck

@dongjinleekr

dongjinleekr commented Nov 30, 2019

Copy link
Copy Markdown
Contributor Author

@omkreddy Could you have a look when you are free?

omkreddy
omkreddy approved these changes Nov 30, 2019
View reviewed changes

@omkreddy omkreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dongjinleekr Thanks for the PR. LGTM.

@omkreddy omkreddy merged commit 6649b7a into apache:trunk Nov 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@omkreddy omkreddy omkreddy 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.

2 participants

@dongjinleekr @omkreddy