On Fri, Apr 29, 2022 at 06:35:52PM +0000, Cary Huang wrote:
> The patch applied and tested fine. I think for this kind of exception, it really is up to the administrator how
he/sheshould proceed to resolve depending on his/her business application. Leaving things configurable by the user is
generallya nice and modest change. I also like that you leave the parameters as enum entry so it is possible to extend
otherpossible actions such as automatically adjust to match the new value.
Thanks for reviewing! Do you think this patch can be marked as
ready-for-committer?
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com
