kamalcph commented on PR #20913:
URL: https://github.com/apache/kafka/pull/20913#issuecomment-4404504887

   > if we need to use -1 for remote.copy.lag.bytes. thus consider -1 is max 
delay lag as the difinition. it will be somehow confuse.
   There is no confusion. The segment upload is allowed when either one of the 
criteria (size / time) gets breached. 
   
   > What's more. for the case which user set both size and time based local 
retention. user if only set one copy lag and forget another one. the final 
result is upload at once. this will avoid user fail to delete the segements 
based on two policies.
   
   "user if only set one copy lag and forget another one. the final result is 
upload at once." -- this is a misconfiguration from the user-end, we should not 
optimize for misconfigured values. 


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

Reply via email to