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

Posted to notifications@geode.apache.org by "moleske (GitHub)" <gi...@apache.org> on 2019/09/30 02:22:59 UTC

[GitHub] [geode-native] moleske commented on pull request #526: GEODE-7207: Fix remaining LGTM "Error" level alerts

Was totally just fixing complaints, didn't actually look at the substance 😞 

Apparently we are never actually using the list the to stop individual servers, that should probably be the real fix instead of just deleting the list.

For record keeping, LGTM was pointing out that we added to the list of servers, but never actually retrieved any information from that list later.

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