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

Posted to notifications@geode.apache.org by "jinmeiliao (GitHub)" <gi...@apache.org> on 2018/10/31 21:41:04 UTC

[GitHub] [geode] jinmeiliao commented on issue #2751: GEODE-5960: Add test to verify JDBC gfsh commands load properly

I am not sure what this test is for. the commandManager actually uses the package scanner to scan the classpath to compose its list of commands, so the test is just reiterates what the commandManager is doing. So if a command did not implement CommandMarker, this test would still pass, I believe.

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