edoardocomar commented on PR #15910: URL: https://github.com/apache/kafka/pull/15910#issuecomment-2104835965
Hi @gharris1727 we're now handling errors in loading the Checkpoints topic. (we still have to add unit tests) Specifically we tested with the not authorized to read case - which the existing KafkaBasedLog was not handling well. At this current stage the task start would fail, which to us seems an improvement as it is detectable and actionable (expecting the change to be noted in the release notes). This looks to us a better behavior than reverting to the old one in case of failure, as maintaining and testing two modes of operation seems too complex. Do you still think we need a KIP - to introduce yet another config to choose between the old behavior (default) and the new one (arguably better in the eyes of this PR authors ...) ? -- 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
