Gargi-jais11 commented on code in PR #8459:
URL: https://github.com/apache/ozone/pull/8459#discussion_r2153637526
##########
hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/keyvalue/TestTarContainerPacker.java:
##########
@@ -72,7 +72,7 @@ public class TestTarContainerPacker {
private static final String TEST_CHUNK_FILE_CONTENT = "This is a chunk";
private static final String TEST_DESCRIPTOR_FILE_CONTENT =
"!<KeyValueContainerData>\n" +
- "checksum:
2215d39f2ae1de89fec837d18dc6387d8cba22fb5943cf4616f80c4b34e2edfe\n" +
+ "checksum:
5e4bea7286f96d88a5b3a745011ff9e4281a5221bfe564413215cd85871dcfd8\n" +
Review Comment:
The checksum change was necessary because the point of checksum verification
shifted. The checksum embedded in `TEST_DESCRIPTOR_FILE_CONTENT` must now
accurately reflect the content of that string before any state changes occur
during the unpackContainerData process. So the new checksum `5e4bea...`
correctly corresponds to the TEST_DESCRIPTOR_FILE_CONTENT which includes state:
CLOSED.
--
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]
