dlmarion commented on PR #4453: URL: https://github.com/apache/accumulo/pull/4453#issuecomment-2052254364
>We could also improve the logging around the memory check to make the issue more obvious but just not letting it easily happen in the first place makes sense to me. > But if a user specifically tries to set a lower value on a system table like metadata then maybe we should throw an exception and prevent that config change as it would just be ignored anyways. I'm wondering if instead of changing the code to throw an exception, we just document the behavior change in Property.java. Something like `Values lower than the default value are ignored for system tables.` -- 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]
