[GitHub] [kafka] cadonna commented on pull request #13269: KAFKA-12634 enforce checkpoint after restoration

["cadonna (via GitHub)" <gi...@apache.org>]

cadonna commented on PR #13269:
URL: https://github.com/apache/kafka/pull/13269#issuecomment-1467588143

   > Sorry for jumping into this PR late @philipnee , for this ticket, we have fixed it in the new state updater thread ([KAFKA-10199](https://issues.apache.org/jira/browse/KAFKA-10199). And we are expecting to enable the state updater soon. So I'd suggest we do not try to fix it inside the main stream thread any more. WDYT?
   
   @guozhangwang 
   Do you think this change is too risky? 
   I do not know how long "soon" means. At the moment, it seems It will still take us some time until we will enable the state updater by default. Even then, we will keep the option to switch back to the old restoration for a couple of releases. 
   So, if we think this PR is not too risky, I would go for it. 
   Maybe I am a bit out of the loop currently and I am missing something.  
   


-- 
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-unsubscribe@kafka.apache.org

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