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

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

vvcephei commented on pull request #6416: KAFKA-8040: Streams handle 
initTransactions timeout
URL: https://github.com/apache/kafka/pull/6416
 
 
   As of 2.0, Producer.initTransactions may throw a TimeoutException, which is 
retriable. Streams should retry instead of crashing when we encounter this 
exception
   
   Reviewers: Guozhang Wang <wangg...@gmail.com>, Matthias J. Sax 
<mj...@apache.org>,  Bill Bejeck <bbej...@gmail.com>
   
   Cherry-pick of #6372
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Streams needs to retry initTransactions
> ---------------------------------------
>
>                 Key: KAFKA-8040
>                 URL: https://issues.apache.org/jira/browse/KAFKA-8040
>             Project: Kafka
>          Issue Type: Improvement
>          Components: streams
>    Affects Versions: 2.0.0, 2.0.1, 2.1.0, 2.2.0, 2.1.1
>            Reporter: John Roesler
>            Assignee: John Roesler
>            Priority: Critical
>             Fix For: 2.2.0, 2.0.2, 2.1.2
>
>
> Following on KAFKA-6446, Streams needs to handle the new behavior.
> `initTxn` can throw TimeoutException now: default `MAX_BLOCK_MS_CONFIG` in 
> producer is 60 seconds, so I ([~guozhang]) think just wrapping it as 
> StreamsException should be reasonable, similar to what we do for 
> `producer#send`'s TimeoutException 
> ([https://github.com/apache/kafka/blob/trunk/streams/src/main/java/org/apache/kafka/streams/processor/internals/RecordCollectorImpl.java#L220-L225]
>  ).
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to