Skip to content

Commit 9c3b57c

Browse files
authored
[HUDI-5063] Enabling run time stats to be serialized with commit metadata (#7006)
1 parent 9fd5fc1 commit 9c3b57c

2 files changed

Lines changed: 3 additions & 6 deletions

File tree

hudi-common/src/main/java/org/apache/hudi/common/model/HoodieWriteStat.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@
1818

1919
package org.apache.hudi.common.model;
2020

21-
import com.fasterxml.jackson.annotation.JsonIgnore;
22-
import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
2321
import org.apache.hadoop.fs.Path;
2422
import org.apache.hudi.common.util.JsonUtils;
2523

@@ -31,7 +29,6 @@
3129
/**
3230
* Statistics about a single Hoodie write operation.
3331
*/
34-
@JsonIgnoreProperties(ignoreUnknown = true)
3532
public class HoodieWriteStat implements Serializable {
3633

3734
public static final String NULL_COMMIT = "null";
@@ -163,7 +160,6 @@ public class HoodieWriteStat implements Serializable {
163160
private Long maxEventTime;
164161

165162
@Nullable
166-
@JsonIgnore
167163
private RuntimeStats runtimeStats;
168164

169165
public HoodieWriteStat() {

hudi-common/src/test/java/org/apache/hudi/common/model/TestHoodieCommitMetadata.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,9 @@ public void testPerfStatPresenceInHoodieMetadata() throws Exception {
7777
String serializedCommitMetadata = commitMetadata.toJsonString();
7878
HoodieCommitMetadata metadata =
7979
HoodieCommitMetadata.fromJsonString(serializedCommitMetadata, HoodieCommitMetadata.class);
80-
// Make sure timing metrics are not written to instant file
81-
assertEquals(0, (long) metadata.getTotalScanTime());
80+
assertTrue(commitMetadata.getTotalCreateTime() > 0);
81+
assertTrue(commitMetadata.getTotalUpsertTime() > 0);
82+
assertTrue(commitMetadata.getTotalScanTime() > 0);
8283
assertTrue(metadata.getTotalLogFilesCompacted() > 0);
8384
}
8485

0 commit comments

Comments
 (0)