You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Kenneth Howe (JIRA)" <ji...@apache.org> on 2018/04/20 15:26:01 UTC

[jira] [Resolved] (GEODE-2984) Gfsh command error handling should use exceptions rather than status returns

     [ https://issues.apache.org/jira/browse/GEODE-2984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kenneth Howe resolved GEODE-2984.
---------------------------------
    Resolution: Not A Problem

> Gfsh command error handling should use exceptions rather than status returns
> ----------------------------------------------------------------------------
>
>                 Key: GEODE-2984
>                 URL: https://issues.apache.org/jira/browse/GEODE-2984
>             Project: Geode
>          Issue Type: Task
>          Components: gfsh
>            Reporter: Jared Stewart
>            Assignee: Kenneth Howe
>            Priority: Major
>
> Currently, a given gfsh command either returns an InfoResultData (in the happy case) or an ErrorResultData (in the exceptional case).  This requires putting error handling code inside of *each* command separately.   It would be far better to instead allow gfsh commands throw an exception, and to have GfshExecutionStrategy (or somewhere similar) handle wrapping up any exception into an ErrorResultData in a generic fashion.
> It may also be worth looking at the built-in mechanism in Spring Shell for this strategy (see afterThrowingInvocation): http://docs.spring.io/autorepo/docs/spring-shell/1.2.0.M1/api/org/springframework/shell/core/ExecutionProcessor.html



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)