EdColeman commented on issue #3529: URL: https://github.com/apache/accumulo/issues/3529#issuecomment-1604312581
It really seems that isPropertySet is checking if the key exists, not necessarily that the property was set. So far, it looks like any property that we have defined will return true - the check will climb the hierarchy and if not found, it will check the default - which will exist and return true. This is even if the value is null or an empty string as the default value. The only time that isPropertySet would seem to work as excepted would be for custom properties - if a custom property is set, it will return true. If not, there will not be a default, so it will be false. That still leaves the determination if the set value is valid or not up to the caller. -- 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]
