lhotari opened a new pull request #8358:
URL: https://github.com/apache/pulsar/pull/8358


   Fixes #8346
   
   ### Motivation
   
   See #8346, #8345
   
   ### Modifications
   
   Prevent setting invalid retention policy where either size or time limit is 
set to zero while the other limit has a non-zero value.
   The reason for this is that setting either size or time limit to zero will 
effectively disable the retention policy.
   It is confusing for the end-user if it's possible to set a value for the 
other limit while it's ignored when the other limit has
   the value of zero. This might lead to the incorrect assumption that the 
value of 0 has the meaning that the limit isn't effective.
   The validation will provide the Admin API end user a useful error message if 
the validation fails.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to