ctubbsii opened a new pull request, #3788: URL: https://github.com/apache/accumulo/pull/3788
* Simplify ServiceLockData's if conditions in methods that return null * Ensure ServiceLockData.getAddress() returns null when .getAddressString() returns null * Handle null toString() in ServiceLockData.equals() using Objects.equals() * Simplify TabletMetadata.checkServer(), TServerClient.getTabletServerConnection(), ExternalCompactionUtil.findCompactionCoordinator(), LiveTServerSet.checkServer(), and Monitor.fetchGcStatus() by using Optional.map(), Optional.ifPresent(), and Optional.orElse(null) * Improve Monitor.fetchGcStatus() by short-circuiting the warning and returning null results when the address is null, rather than wait for an exception to be thrown while trying to connect to the service using a null address * Update ServiceLockDataTest to ensure getAddress() returns null when it's expected to, instead of throwing a NullPointerException This fixes #3782 -- 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]
