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

Posted to notifications@geode.apache.org by "demery-pivotal (GitHub)" <gi...@apache.org> on 2019/01/30 01:29:18 UTC

[GitHub] [geode] demery-pivotal commented on pull request #3137: GEODE-6334: Create CachePerfStatsTest and cleanup CachePerfStats

This looks like a characterization test (demonstrating what the code currently does) rather than a test of what we really want it to do. Consider mentioning that in a comment. Similar for the other "wraps to negative" tests.

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