You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2020/06/17 06:40:00 UTC
[jira] [Commented] (GEODE-8268) redis ExecutionHandlerContext is
doing more than it should
[ https://issues.apache.org/jira/browse/GEODE-8268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17138136#comment-17138136 ]
ASF GitHub Bot commented on GEODE-8268:
---------------------------------------
dschneider-pivotal merged pull request #5237:
URL: https://github.com/apache/geode/pull/5237
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
> redis ExecutionHandlerContext is doing more than it should
> ----------------------------------------------------------
>
> Key: GEODE-8268
> URL: https://issues.apache.org/jira/browse/GEODE-8268
> Project: Geode
> Issue Type: Improvement
> Components: redis
> Reporter: Darrel Schneider
> Assignee: Darrel Schneider
> Priority: Major
>
> ExecutionHandlerContext is supposed to provide context to redis command executors so that they can execute their specific command. But in a couple of cases (SHUTDOWN and SUBSCRIBE)Â ExecutionHandlerContext has code that is part of the execution of a specific command. That code should be refactored so that the specific command drives the execution of itself.
> Â
--
This message was sent by Atlassian Jira
(v8.3.4#803005)