amrishlal commented on code in PR #9336:
URL: https://github.com/apache/hudi/pull/9336#discussion_r1281234501
##########
hudi-common/src/main/java/org/apache/hudi/common/table/timeline/HoodieTimeline.java:
##########
@@ -413,6 +423,14 @@ static boolean isInRange(String timestamp, String startTs,
String endTs) {
&& HoodieTimeline.compareTimestamps(timestamp,
LESSER_THAN_OR_EQUALS, endTs);
}
+ /**
+ * Return true if specified timestamp is in range [startTs, endTs].
+ */
+ static boolean isInClosedRange(String timestamp, String startTs, String
endTs) {
Review Comment:
Can this use half open range?
--
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]