[GitHub] [accumulo] ctubbsii commented on issue #3486: Add tests for AccumuloConfiguration.isPropertySet

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii commented on issue #3486:
URL: https://github.com/apache/accumulo/issues/3486#issuecomment-1663097151

   > Maybe the `isPropertySet()` checks the property against the return value of `get()` for null so that the two do not diverge accidentally at somepoint.
   
   The point of `isPropertySet()` is completely independent of the null-ness of the return value of `get()`. There's no synchronicity between them to check. They are already divergent. Null or not null just means that's the current value. It says nothing about whether a user set that value or if that's the default. That's why `isPropertySet()` was created... to intentionally keep it independent from the value returned from `get()`.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org