[GitHub] [geode] BenjaminPerryRoss commented on pull request #2918: GEODE-6102: add gfsh destroy data-source

["BenjaminPerryRoss (GitHub)" <gi...@apache.org>]

After looking over this, there is no code path which would allow us to attempt updating cluster config in the event the binding is not found within cluster config.

@jchen21 The reason you might see those two messages together is in the event that there are no servers but cluster config is enabled and has the specified binding in it's list of bindings. In this case the command would update cluster config to remove the binding, but would also return 'no members found' as it wouldn't be able to construct a table of function results (the normal output for a successful destroy command) since no servers were available to run the function.

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