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

Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/02/22 18:23:56 UTC

[GitHub] [geode] kirklund opened pull request #3224: GEODE-6442: Use PID for OS and Process stats if available

If the PID is unavailable for any reason, fall back to using
DistributedSystem ID.


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

[GitHub] [geode] kirklund closed pull request #3224: GEODE-6442: Use PID for OS and Process stats if available

Posted by "kirklund (GitHub)" <gi...@apache.org>.

[ pull request closed by kirklund ]

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

[GitHub] [geode] kirklund commented on issue #3224: GEODE-6442: Use PID for OS and Process stats if available

Posted by "kirklund (GitHub)" <gi...@apache.org>.

IntegrationTest failures:
```
org.apache.geode.codeAnalysis.AnalyzeSerializablesJUnitTest > testSerializables FAILED
    java.lang.AssertionError: New or moved classes----------------------------------------
    org/apache/geode/internal/process/UncheckedPidUnavailableException,true,-4896120572252598765


    If the class is not persisted or sent over the wire add it to the file 
    /home/geode/geode/geode-core/build/resources/integrationTest/org/apache/geode/codeAnalysis/excludedClasses.txt
    Otherwise if this doesn't break backward compatibility, copy the file 
    /home/geode/geode/geode-core/build/integrationTest588/actualSerializables.dat to 
    /home/geode/geode/geode-core/build/resources/main/org/apache/geode/internal/sanctioned-geode-core-serializables.txt.
```

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

[GitHub] [geode] moleske commented on pull request #3224: GEODE-6442: Use PID for OS and Process stats if available

Posted by "moleske (GitHub)" <gi...@apache.org>.

Liking the new tests here

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