haanhvu commented on pull request #3812: URL: https://github.com/apache/activemq-artemis/pull/3812#issuecomment-952122872
> Since the existing functionality uses 'max-disk-usage' as a name, I think something like 'min-disk-...' would be more consistent than 'min-free-disk-bytes' is. I wouldn't put the 'bytes' units in the name, allowing for more friendly unit-specifying strings to also be supplied, e.g. as is supported for the "global-max-size". Agree. I actually thought unit-specified strings would be better too. About the name, does "min-free-disk" sound better? > If allowing for units, a different route and obvious simplification to avoid needing to consider any conflicting 'max usage' and 'min available' configurations would be to instead just extend what the existing 'max-disk-usage' can do and allow it to specify the actual usage amount rather than a percentage of the total. This is of course a simpler way. However, it will force users to make a calculation of the actual usage amount. Also, users will need to update this value every time their disk size increases. With "min-free-disk", users don't have to do those. So I still vote for "min-free-disk" as a better choice. > To handle that I guess I would leave the existing max-disk-usage default to retain existing behaviour, and remove any default for the new 'min available' config. That way you know it was configured explicitly if it is set, and we could choose to document that it takes precedence over the max-disk-usage setting, i.e they can choose to use 'max-disk-usage' or to instead use 'min-disk-...' I think this approach is the optimal one now. I'll implement it. If I'm stuck somewhere, I'll ping you. Do you agree? -- 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]
