You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "dschneider-pivotal (GitHub)" <gi...@apache.org> on 2019/01/08 00:16:27 UTC

[GitHub] [geode] dschneider-pivotal commented on pull request #3055: GEODE-6247: Added suspected exception to ignore list

I think it would be better to change the product to handle this jdk bug (see: https://bugs.openjdk.java.net/browse/JDK-8207200). Kirk had an idea of how to do that. Basically when VMStats50.refresh calls getHeapMemoryUsage catch IllegalArgumentException, log something (once?), and then do the remainder of the refresh method. This prevents us from updating some stats but at least by catching the exception we would do the other stats. A user might use this jdk so it seems like it is worth us handling this unexpected exception.

[ Full content available at: https://github.com/apache/geode/pull/3055 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org