Skip to content

Commit a6685c4

Browse files
committed
chore: remove trivial comments
1 parent 2fe7fde commit a6685c4

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/SnapshotDiffJob.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -341,7 +341,7 @@ public CodecBuffer toCodecBuffer(@Nonnull SnapshotDiffJob object,
341341
@Override
342342
public SnapshotDiffJob fromCodecBuffer(@Nonnull CodecBuffer buffer)
343343
throws CodecException {
344-
// Direct protobuf parsing from InputStream to avoid byte array copy
344+
345345
try (java.io.InputStream in = buffer.getInputStream()) {
346346
SnapshotDiffJobProto proto = SnapshotDiffJobProto.parseFrom(in);
347347
return SnapshotDiffJob.getFromProtoBuf(proto);

hadoop-ozone/common/src/test/java/org/apache/hadoop/ozone/om/helpers/TestOmSnapshotDiffJobCodec.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ public class TestOmSnapshotDiffJobCodec {
3737

3838
@Test
3939
public void testOldJsonSerializedDataCanBeReadByNewCodec() throws Exception {
40-
// Step 1: Construct a SnapshotDiffJob instance
4140
SnapshotDiffJob original = new SnapshotDiffJob(
4241
123456789L,
4342
"job-001",
@@ -53,13 +52,10 @@ public void testOldJsonSerializedDataCanBeReadByNewCodec() throws Exception {
5352
0.0
5453
);
5554

56-
// Step 2: Serialize using the old Jackson-based codec
5755
byte[] oldFormatData = oldCodec.toPersistedFormatImpl(original);
5856

59-
// Step 3: Deserialize using the new default codec (with Protobuf + JSON fallback)
6057
SnapshotDiffJob parsed = newCodec.fromPersistedFormatImpl(oldFormatData);
6158

62-
// Step 4: Verify critical fields remain consistent after round-trip
6359
assertEquals(original.getJobId(), parsed.getJobId());
6460
assertEquals(original.getStatus(), parsed.getStatus());
6561
assertEquals(original.getVolume(), parsed.getVolume());

0 commit comments

Comments
 (0)