Skip to content

HOTFIX: ChangeLoggingKeyValueStore.name() returns null#2397

Closed
dguy wants to merge 1 commit intoapache:trunkfrom
dguy:hotfix-npe-state-store
Closed

HOTFIX: ChangeLoggingKeyValueStore.name() returns null#2397
dguy wants to merge 1 commit intoapache:trunkfrom
dguy:hotfix-npe-state-store

Conversation

@dguy
Copy link
Copy Markdown
Contributor

@dguy dguy commented Jan 18, 2017

This class doesn't need to override this method as it is handled appropriately by the super class

@dguy
Copy link
Copy Markdown
Contributor Author

dguy commented Jan 18, 2017

@guozhangwang, @enothereska, @mjsax - this is needed for 0.10.1

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 18, 2017

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

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 18, 2017

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

@asfbot
Copy link
Copy Markdown

asfbot commented Jan 18, 2017

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

@guozhangwang
Copy link
Copy Markdown
Contributor

@dguy You mean 0.10.2 right?

asfgit pushed a commit that referenced this pull request Jan 18, 2017
This class doesn't need to override this method as it is handled appropriately by the super class

Author: Damian Guy <damian.guy@gmail.com>

Reviewers: Guozhang Wang <wangguoz@gmail.com>

Closes #2397 from dguy/hotfix-npe-state-store

(cherry picked from commit 961ebca)
Signed-off-by: Guozhang Wang <wangguoz@gmail.com>
@guozhangwang
Copy link
Copy Markdown
Contributor

Merged to 0.10.2. and trunk

@asfgit asfgit closed this in 961ebca Jan 18, 2017
@dguy
Copy link
Copy Markdown
Contributor Author

dguy commented Jan 18, 2017

yeah - i meant 0.10.2

@dguy dguy deleted the hotfix-npe-state-store branch January 19, 2017 11:40
soenkeliebau pushed a commit to soenkeliebau/kafka that referenced this pull request Feb 7, 2017
This class doesn't need to override this method as it is handled appropriately by the super class

Author: Damian Guy <damian.guy@gmail.com>

Reviewers: Guozhang Wang <wangguoz@gmail.com>

Closes apache#2397 from dguy/hotfix-npe-state-store
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