poorbarcode commented on code in PR #15929:
URL: https://github.com/apache/pulsar/pull/15929#discussion_r889846336
##########
pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/RetentionPolicies.java:
##########
@@ -28,24 +28,33 @@
* Infinite retention can be achieved by setting both time and size limits to
`-1`.
*/
public class RetentionPolicies {
- private int retentionTimeInMinutes;
- private long retentionSizeInMB;
+
+ public static final int DEFAULT_RETENTION_TIME_IN_MINUTES = 0;
+
+ public static final long DEFAULT_RETENTION_SIZE_IN_MB = 0;
+
+ private Integer retentionTimeInMinutes;
+ private Long retentionSizeInMB;
Review Comment:
There are no compatibility issues and it can be set to 0. But this PR does
not solve the problem that only one property is set correctly, so I will close
this PR.
--
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]
