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/she should proceed to resolve 
> depending on his/her business application. Leaving things configurable by the 
> user is generally a nice and modest change. I also like that you leave the 
> parameters as enum entry so it is possible to extend other possible 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


Reply via email to