lianetm commented on PR #17440:
URL: https://github.com/apache/kafka/pull/17440#issuecomment-2517959270

   > Yes, I believe this approach can solve the problem, but it does feel a bit 
frustrating since we spent quite some time unifying the metadata error 
propagation method. Perhaps we can create a task later to unify the error 
propagation behavior of both types of consumers.
   
   Well this PR achieved unifying it within the KafkaConsumer, that's a great 
step! Unifying it between the 2 consumers is a different story, worth exploring 
but the challenge truly is that the position management is quite different 
between the 2 (which is the main point of friction we had in the Async 
consumer, where we needed to detect these failures when updating positions from 
poll and position, via the same event). 


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