siying opened a new pull request, #50480:
URL: https://github.com/apache/spark/pull/50480

   ### What changes were proposed in this pull request?
   When State Store Checkpoint format V2 is used, we always read back 
checkpoint ID from commit log, rather than when commit log matches offset log.
   
   ### Why are the changes needed?
   Right now, there is a bug of reading checkpoint ID from commit log when the 
query restarts. If the offset log is ahead of commit log, it doesn't read it, 
and the tasks won't have checkpoint ID to recover from and the query will fail.
   
   
   ### Does this PR introduce _any_ user-facing change?
   No.
   
   
   ### How was this patch tested?
   Add a unit test that will fail without the fix.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No.
   


-- 
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: reviews-unsubscr...@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to