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

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

GitHub user hachikuji opened a pull request:

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

    KAFKA-5248: Remove unused/unneeded retention time in TxnOffsetCommitRequest

    

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

    $ git pull https://github.com/hachikuji/kafka KAFKA-5248

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

    https://github.com/apache/kafka/pull/3058.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 #3058
    
----
commit 9290f30b60d1264697494aaacda2d485ee6b237c
Author: Jason Gustafson <ja...@confluent.io>
Date:   2017-05-15T18:13:11Z

    KAFKA-5248: Remove unused/unneeded retention time in TxnOffsetCommitRequest

----


> Remove retention time from TxnOffsetCommit RPC
> ----------------------------------------------
>
>                 Key: KAFKA-5248
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5248
>             Project: Kafka
>          Issue Type: Sub-task
>          Components: clients, core, producer 
>            Reporter: Jason Gustafson
>            Assignee: Jason Gustafson
>             Fix For: 0.11.0.0
>
>
> We added offset retention time because OffsetCommitRequest had it. However, 
> the new consumer has never exposed this and we have no plan of exposing it in 
> the producer, so we may as well remove it. If we need it later, we can bump 
> the protocol.



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

Reply via email to