ChenSammi commented on code in PR #5663:
URL: https://github.com/apache/ozone/pull/5663#discussion_r1467319224
##########
hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/storage/BlockOutputStream.java:
##########
@@ -80,6 +82,12 @@ public class BlockOutputStream extends OutputStream {
LoggerFactory.getLogger(BlockOutputStream.class);
public static final String EXCEPTION_MSG =
"Unexpected Storage Container Exception: ";
+ public static final String INCREMENTAL_CHUNK_LIST = "incremental";
+ public static final KeyValue INCREMENTAL_CHUNK_LIST_KV =
+ KeyValue.newBuilder().setKey(INCREMENTAL_CHUNK_LIST).build();
+ public static final String FULL_CHUNK = "full";
+ public static final KeyValue FULL_CHUNK_KV =
+ KeyValue.newBuilder().setKey(FULL_CHUNK).build();
Review Comment:
Can you put INCREMENTAL_CHUNK_LIST, FULL_CHUNK in some common places. I
remember there is one definition too in BlockManagerImpl.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
