diff --git a/hudi-common/src/main/java/org/apache/hudi/common/model/HoodieWriteStat.java b/hudi-common/src/main/java/org/apache/hudi/common/model/HoodieWriteStat.java index 91637102f0404..6c1efc1561210 100644 --- a/hudi-common/src/main/java/org/apache/hudi/common/model/HoodieWriteStat.java +++ b/hudi-common/src/main/java/org/apache/hudi/common/model/HoodieWriteStat.java @@ -18,8 +18,6 @@ package org.apache.hudi.common.model; -import com.fasterxml.jackson.annotation.JsonIgnore; -import com.fasterxml.jackson.annotation.JsonIgnoreProperties; import org.apache.hadoop.fs.Path; import org.apache.hudi.common.util.JsonUtils; @@ -31,7 +29,6 @@ /** * Statistics about a single Hoodie write operation. */ -@JsonIgnoreProperties(ignoreUnknown = true) public class HoodieWriteStat implements Serializable { public static final String NULL_COMMIT = "null"; @@ -163,7 +160,6 @@ public class HoodieWriteStat implements Serializable { private Long maxEventTime; @Nullable - @JsonIgnore private RuntimeStats runtimeStats; public HoodieWriteStat() { diff --git a/hudi-common/src/test/java/org/apache/hudi/common/model/TestHoodieCommitMetadata.java b/hudi-common/src/test/java/org/apache/hudi/common/model/TestHoodieCommitMetadata.java index b0b59f8cbcb8b..e8c159540a3d3 100644 --- a/hudi-common/src/test/java/org/apache/hudi/common/model/TestHoodieCommitMetadata.java +++ b/hudi-common/src/test/java/org/apache/hudi/common/model/TestHoodieCommitMetadata.java @@ -77,8 +77,9 @@ public void testPerfStatPresenceInHoodieMetadata() throws Exception { String serializedCommitMetadata = commitMetadata.toJsonString(); HoodieCommitMetadata metadata = HoodieCommitMetadata.fromJsonString(serializedCommitMetadata, HoodieCommitMetadata.class); - // Make sure timing metrics are not written to instant file - assertEquals(0, (long) metadata.getTotalScanTime()); + assertTrue(commitMetadata.getTotalCreateTime() > 0); + assertTrue(commitMetadata.getTotalUpsertTime() > 0); + assertTrue(commitMetadata.getTotalScanTime() > 0); assertTrue(metadata.getTotalLogFilesCompacted() > 0); }