adixitconfluent commented on code in PR #20124: URL: https://github.com/apache/kafka/pull/20124#discussion_r2195525999
########## core/src/main/java/kafka/server/share/SharePartition.java: ########## @@ -3016,6 +3023,9 @@ static final class InFlightState { private InFlightState rollbackState; // The timer task for the acquisition lock timeout. private AcquisitionLockTimerTask acquisitionLockTimeoutTask; + // The boolean determines if the record has achieved a final state of ARCHIVED from which it cannot transition + // to any other state. This could happen because of LSO movement etc. + private boolean isMarkedArchived = false; Review Comment: Agreed, I can use `isTerminalState`, sounds better to me -- 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