keith-turner commented on issue #3909:
URL: https://github.com/apache/accumulo/issues/3909#issuecomment-1787685193
> We likely need to add some logic to PropUtil.validateProperties for this.
However, without annotating which properties accept json, we might have to
hardcode the list for now.
It would be nice to validate these properties at set time, but not sure its
possible. Would need to overcome the following in validation
* The compaction service config spans multiple properties.
* Validating JSON would not be enough and could break plugins that are not
the default. A planner plugin implementation may not take json, or it could
take json but expect a very specific schema.
Opened #3912 to make the system behave better when there is bad config as
identified in this issue. Need to do some more general follow on work to
address per table compaction dispatchers having config that makes them throw an
exception, that may cause a similar log storm.
--
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]
