prashantwason commented on a change in pull request #2701:
URL: https://github.com/apache/hudi/pull/2701#discussion_r663169332
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/table/timeline/TimelineLayout.java
##########
@@ -74,5 +82,26 @@ public static TimelineLayout getLayout(TimelineLayoutVersion
version) {
return y;
}).get());
}
+
+ @Override
+ public Stream<HoodieInstant> sortHoodieInstants(Stream<HoodieInstant>
instantStream) {
+ return instantStream.sorted();
+ }
+ }
+
+ /**
+ * Table Layout where state transitions are managed by creating new files
with END_INSTANT_TIME.
+ */
+ private static class TimelineLayoutV2 extends TimelineLayoutV1 {
+
+ @Override
Review comment:
Is this required if it simply calls super method?
--
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]
