[GitHub] [geode] kirklund commented on pull request #4020: GEODE-7170: Move InetAddressUtils to internal.net package

2019-09-18 Thread GitHub
I added InetAddressUtilsWithLogging which wraps two of the utils with versions that log call stack. I also updated all uses of both InetAddressUtils classes to use `import static` [ Full content available at: https://github.com/apache/geode/pull/4020 ] This message was relayed via

[GitHub] [geode] kirklund commented on pull request #4020: GEODE-7170: Move InetAddressUtils to internal.net package

2019-09-09 Thread GitHub
No one is using Admin API. It was deprecated before Apache Geode 1.0.0. I removed the logging of stack traces mainly because it's bad form for simple low-level utils like this to perform logging. The main reason to move this class out of admin.internal is because we're going to delete