sudeshwasnik commented on PR #12784:
URL: https://github.com/apache/kafka/pull/12784#issuecomment-1295033345

   @C0urante @vamossagar12 quick dumb question - if there is an Async failure 
(detected after offsets are dispatched to producer), won't 
producer.commitTransaction also throw exception eventually? which gets captured 
in `flushError` (atleast that's what I expect, hence curious). 
   If so, is there a case where we require Async error tracking in offsets sent 
to producer separately? 
   
   Although I agree, we should anyways. And +1 to the suggestion! I'll try to 
add tests for Async failures (which is different from 
`FAIL_TRANSACTION_COMMIT`).  
   thansk for the review! 


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to