Skip to content

HOTFIX: ClassCastException in Request logging#2571

Closed
original-brownbear wants to merge 1 commit intoapache:trunkfrom
original-brownbear:fix-logging-err-response
Closed

HOTFIX: ClassCastException in Request logging#2571
original-brownbear wants to merge 1 commit intoapache:trunkfrom
original-brownbear:fix-logging-err-response

Conversation

@original-brownbear
Copy link
Copy Markdown
Member

Comming from here

Fixed ClassCastException resulting from missing type hint in request logging.

@original-brownbear
Copy link
Copy Markdown
Member Author

@ijuma fyi :)

@hachikuji
Copy link
Copy Markdown
Contributor

LGTM, thanks for the PR. Thought I checked for other usages, but looks like I missed this one.

@asfgit asfgit closed this in d24d932 Feb 18, 2017
@original-brownbear original-brownbear deleted the fix-logging-err-response branch February 18, 2017 18:06
@asfbot
Copy link
Copy Markdown

asfbot commented Feb 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/1762/
Test PASSed (JDK 8 and Scala 2.11).

@asfbot
Copy link
Copy Markdown

asfbot commented Feb 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/1759/
Test PASSed (JDK 7 and Scala 2.10).

@asfbot
Copy link
Copy Markdown

asfbot commented Feb 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/1760/
Test FAILed (JDK 8 and Scala 2.12).

hachikuji pushed a commit to confluentinc/kafka that referenced this pull request Feb 23, 2017
Fixed ClassCastException resulting from missing type hint in request logging.

Author: Armin Braun <me@obrown.io>

Reviewers: Jason Gustafson <jason@confluent.io>

Closes apache#2571 from original-brownbear/fix-logging-err-response
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