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

Posted to notifications@geode.apache.org by "jchen21 (GitHub)" <gi...@apache.org> on 2019/02/12 01:10:48 UTC

[GitHub] [geode] jchen21 commented on pull request #3175: GEODE-6365: Add server group support for JDBC List Mapping and Destroy Mapping Commands

Instead of having the null check here, can we just pass `TEST_REGION` as parameter wherever appropriate? e.g. `verifyQueueRemoved(cache, null, false);` becomes `verifyQueueRemoved(cache, TEST_REGION, false);`. This makes the code easier to read and understand.

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