xichen01 commented on code in PR #8953:
URL: https://github.com/apache/ozone/pull/8953#discussion_r2297060274
##########
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/OmLCExpiration.java:
##########
@@ -38,6 +38,7 @@
public final class OmLCExpiration implements OmLCAction {
private final Integer days;
private final String date;
+ private final ZonedDateTime createDate;
Review Comment:
I have directly used the `OmLifecycleConfiguration#creationTime`, and this
is already a persist field, and we do not use it previous.
A Lifecycle may contains multiple Rule, both these Rule should have a same
creation time, so put the creationTime on the OmLifecycleConfiguration is
appropriate.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
