[jira] [Commented] (KAFKA-5128) TransactionCoordinator - Check inter broker protocol and message format and raise errors if incompatible

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/KAFKA-5128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16017561#comment-16017561 ] 

ASF GitHub Bot commented on KAFKA-5128:
---------------------------------------

GitHub user dguy opened a pull request:

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

    KAFKA-5128: check inter broker and message format versions in transactional methods [WIP]

    Add check in `KafkaApis` that the inter broker protocol and message format versions are at least `KAFKA_0_11_0_IV0`, i.e., supporting transactions 

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

    $ git pull https://github.com/dguy/kafka kafka-5128

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

    https://github.com/apache/kafka/pull/3103.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 #3103
    
----
commit 75846aee5a945ea13c08856fc12ffc4ff62f7e45
Author: Damian Guy <da...@gmail.com>
Date:   2017-05-19T15:36:22Z

    check api version in transactional methods

----


> TransactionCoordinator - Check inter broker protocol and message format and raise errors if incompatible
> --------------------------------------------------------------------------------------------------------
>
>                 Key: KAFKA-5128
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5128
>             Project: Kafka
>          Issue Type: Sub-task
>            Reporter: Damian Guy
>            Assignee: Damian Guy
>            Priority: Blocker
>
> In {{TransactionStateManager}} and {{InterBrokerSendThread}} we need to check if the message format supports transactions and error out if it doesn't



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)