[GitHub] kafka pull request #3673: MINOR: Consolidate broker request/response handlin...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/kafka/pull/3673


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] kafka pull request #3673: MINOR: Consolidate broker request/response handlin...

[hachikuji]

GitHub user hachikuji opened a pull request:

https://github.com/apache/kafka/pull/3673

MINOR: Consolidate broker request/response handling

This patch contains a few small improvements to make request/response 
handling more consistent. Primarily it consolidates request/response 
serialization logic so that `SaslServerAuthenticator` and `KafkaApis` follow 
the same path. It also reduces the amount of custom logic needed to handle 
unsupported versions of the ApiVersions requests.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/hachikuji/kafka consolidate-response-handling

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/kafka/pull/3673.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #3673


commit bbbeb17ed5aa4fcd5ca52bb98564dafdd854bb94
Author: Jason Gustafson 
Date:   2017-08-04T21:58:11Z

MINOR: Consolidate broker request/response handling




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---