keith-turner commented on PR #5365:
URL: https://github.com/apache/accumulo/pull/5365#issuecomment-2691546144
> Okay, after looking more closely, I can see now why it was being done this
way... because it's relying on some of the private methods for normalization of
URIs to do some of the validation. This is trickier than I first thought.
Another reason I structured the code the way it is was to avoid making
assumptions about exceptions thrown in another class. The code calls some
private methods in the same class that throw a specific exception that is
catches.
I have not figured out a good way to do it, but would like to somehow like
to accomplish the following goals.
* Be able to validate volumes independently of creating a server process if
possible. Like validate config in a stand alone utility.
* Support the concept that there is no default value but to be valid config
for a system it must be set to a list of one ore more URIs.
* If possible have the validation code reside in Property and PropertyType
instead of off at some random place in the code.
--
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]
