davehagman commented on a change in pull request #3824:
URL: https://github.com/apache/hudi/pull/3824#discussion_r736770187



##########
File path: 
hudi-common/src/main/java/org/apache/hudi/common/table/timeline/HoodieActiveTimeline.java
##########
@@ -59,7 +59,7 @@
  */
 public class HoodieActiveTimeline extends HoodieDefaultTimeline {
 
-  public static final SimpleDateFormat COMMIT_FORMATTER = new 
SimpleDateFormat("yyyyMMddHHmmss");
+  public static final SimpleDateFormat COMMIT_FORMATTER = new 
SimpleDateFormat("yyyyMMddHHmmssSSS");

Review comment:
       All logic for instant parsing has been brought into the 
`HoodieActiveTimeline` class and all dependent code is now using this 
consistent interface. All tests are also passing so we're looking good here 👍 




-- 
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]


Reply via email to