satishd commented on a change in pull request #10218:
URL: https://github.com/apache/kafka/pull/10218#discussion_r607556778
##########
File path:
clients/src/main/java/org/apache/kafka/server/log/remote/storage/RemoteLogSegmentState.java
##########
@@ -21,14 +21,16 @@
import java.util.Arrays;
import java.util.Collections;
import java.util.Map;
+import java.util.Objects;
import java.util.function.Function;
import java.util.stream.Collectors;
/**
- * It indicates the state of the remote log segment. This will be based on the
action executed on this
+ * This enum indicates the state of the remote log segment. This will be based
on the action executed on this
* segment by the remote log service implementation.
* <p>
- * It goes through the below state transitions.
+ * It goes through the below state transitions. Self transition is treated as
valid. This allows updating with the
+ * same state in case of retries and failover.
Review comment:
Looks like autoformatter changed it.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
