[ 
https://issues.apache.org/jira/browse/GEODE-10216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17517757#comment-17517757
 ] 

ASF subversion and git services commented on GEODE-10216:
---------------------------------------------------------

Commit de91e330525b7e87a1bd20bb162601fe6e2301f6 in geode's branch 
refs/heads/develop from Hale Bales
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=de91e33052 ]

GEODE-10216: Add test for existence of VM stats (#7551)

When we started running with JDK 11, some users noticed that a few stats had 
gone missing. There was a one line fix in test.gradle that was added as part of 
a bigger effort, but the change was not tested. This adds a test for the 
existence of the VM stats so that none of the stats go missing in the future.

also changes all the assertions to use assertj.

> Add test for existence of VM stats
> ----------------------------------
>
>                 Key: GEODE-10216
>                 URL: https://issues.apache.org/jira/browse/GEODE-10216
>             Project: Geode
>          Issue Type: Test
>          Components: statistics
>    Affects Versions: 1.15.0
>            Reporter: Hale Bales
>            Assignee: Hale Bales
>            Priority: Major
>              Labels: pull-request-available
>
> When we started running with JDK 11, a couple of stats went missing (fdsOpen 
> and fdLimit). This was resolved by adding the following line to test.gradle:
> {code:java}
> "--add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED",
> {code}
> This change was not tested. Add a test so that if something similar happens 
> in the future, we will know about it before a customer has an issue.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to