Skip to content

KAFKA-5608: Follow-up to fix potential NPE and clarify method name#3553

Closed
ijuma wants to merge 1 commit intoapache:trunkfrom
ijuma:kafka-5608-follow-up
Closed

KAFKA-5608: Follow-up to fix potential NPE and clarify method name#3553
ijuma wants to merge 1 commit intoapache:trunkfrom
ijuma:kafka-5608-follow-up

Conversation

@ijuma
Copy link
Copy Markdown
Member

@ijuma ijuma commented Jul 20, 2017

No description provided.

@ijuma
Copy link
Copy Markdown
Member Author

ijuma commented Jul 20, 2017

@asfgit
Copy link
Copy Markdown

asfgit commented Jul 20, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6179/
Test PASSed (JDK 8 and Scala 2.12).

@asfgit
Copy link
Copy Markdown

asfgit commented Jul 20, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6195/
Test PASSed (JDK 7 and Scala 2.11).

Copy link
Copy Markdown
Contributor

@ewencp ewencp left a comment

Choose a reason for hiding this comment

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

LGTM

@ijuma
Copy link
Copy Markdown
Member Author

ijuma commented Jul 21, 2017

Merged to trunk and 0.11.0.

asfgit pushed a commit that referenced this pull request Jul 21, 2017
Author: Ismael Juma <ismael@juma.me.uk>

Reviewers: Ewen Cheslack-Postava <me@ewencp.org>

Closes #3553 from ijuma/kafka-5608-follow-up

(cherry picked from commit 08b775c)
Signed-off-by: Ismael Juma <ismael@juma.me.uk>
@asfgit asfgit closed this in 08b775c Jul 21, 2017
@ijuma ijuma deleted the kafka-5608-follow-up branch August 22, 2017 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants