[jira] [Commented] (GEODE-287) Remove old gfsh code

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-287?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14725670#comment-14725670 ] 

ASF subversion and git services commented on GEODE-287:
-------------------------------------------------------

Commit 49d99d4e5d97ecc682e9929251ab959ff7307e7a in incubator-geode's branch refs/heads/feature/GEODE-77 from [~dschneider]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;h=49d99d4 ]

GEODE-287: Remove old gfsh code

This commit is just file deletes except for one unit test
that had an unused import of one of the deleted classes.


> Remove old gfsh code
> --------------------
>
>                 Key: GEODE-287
>                 URL: https://issues.apache.org/jira/browse/GEODE-287
>             Project: Geode
>          Issue Type: Improvement
>            Reporter: Darrel Schneider
>            Assignee: Darrel Schneider
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> An old version of gfsh is in geode in these packages: com.gemstone.gemfire.internal.tools.gfsh.*
> This old version uses deprecated apis that we are trying to remove (see GEODE-72). So this old gfsh code blocks the removal of the deprecated apis.
> One option is to just remove this old gfsh code. Now that we have a new gfsh tool that we support keeping the old code in the open source project does not make sense.
> Another option is to change the old gfsh code to use non-deprecated apis. But I don't know what kind of test coverage exists for the old gfsh so these changes may break it without being detected.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)