yihua commented on code in PR #13056:
URL: https://github.com/apache/hudi/pull/13056#discussion_r2020037468
##########
hudi-client/hudi-client-common/src/test/java/org/apache/hudi/metrics/TestHoodieMetrics.java:
##########
@@ -295,8 +295,11 @@ public void testTimerCtxandGauges() throws
InterruptedException {
HoodieInstant instant008 =
INSTANT_GENERATOR.createNewInstant(HoodieInstant.State.INFLIGHT,
HoodieTimeline.COMPACTION_ACTION, "1008");
HoodieInstant instant0012 =
INSTANT_GENERATOR.createNewInstant(HoodieInstant.State.REQUESTED,
HoodieTimeline.COMPACTION_ACTION, "10012");
HoodieInstant instant0014 =
INSTANT_GENERATOR.createNewInstant(HoodieInstant.State.INFLIGHT,
HoodieTimeline.COMPACTION_ACTION, "10014");
+ HoodieInstant longInstant =
INSTANT_GENERATOR.createNewInstant(HoodieInstant.State.INFLIGHT,
HoodieTimeline.COMPACTION_ACTION, "20250329154030600010001");
Review Comment:
What is the format of this timestamp? Could we document this in javadocs
for MDT in table version 6?
```
20250329154030600010001
date |hhmmssSSS| should we only have 001 as the suffix?
```
--
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: commits-unsubscr...@hudi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
