rakeshadr commented on code in PR #3201:
URL: https://github.com/apache/ozone/pull/3201#discussion_r871638969
##########
hadoop-ozone/interface-client/src/main/proto/OmClientProtocol.proto:
##########
@@ -810,6 +810,41 @@ message KeyLocationList {
optional bool isMultipartKey = 4 [default = false];
}
+/**
+ * Checksum algorithms/types used in Ozone
+ * Make sure this enum's integer values match enum values' id properties
defined
+ * in org.apache.hadoop.util.DataChecksum.Type
+ */
+enum ChecksumTypeProto {
+ CHECKSUM_NULL = 0;
+ CHECKSUM_CRC32 = 1;
+ CHECKSUM_CRC32C = 2;
+}
+
+enum FileChecksumTypeProto {
+ MD5CRC = 1; // BlockChecksum obtained by taking the MD5 digest of chunk CRCs
+ COMPOSITE_CRC = 2; // Chunk-independent CRC, optionally striped
+}
+
+message CompositeCrcFileChecksumProto {
+ required ChecksumTypeProto checksumType = 1;
+ required uint32 bytesPerCrc = 2;
+ required uint32 crc = 3;
+}
+
+message MD5MD5Crc32FileChecksumProto {
+ required ChecksumTypeProto checksumType = 1;
+ required uint32 bytesPerCRC = 2;
+ required uint64 crcPerBlock = 3;
+ required bytes md5 = 4;
+}
+
+message FileChecksumProto {
+ required FileChecksumTypeProto checksumType = 1;
Review Comment:
@jojochuang I'm OK to raise a separate unit test task and cover all these
tests in that. Please create a sub-task, then we can merge this PR and move
ahead to the next task.
--
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]
