[jira] [Commented] (GEODE-3751) Refactor: ClientProtocolService, ClientStreamProcessor

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

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

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

Commit 033757f5569126ad84ee4badf2c283481ec205b4 in geode's branch refs/heads/develop from [~WireBaron]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=033757f ]

GEODE-3751: a single place for client protocol loading, logic.

* Load a single service, `ClientProtocolService`.
  - (well, authenticators are still separate).
* Move several interfaces to `geode-protobuf` from core.
* Client protocol state and statistics are now stored on the new
  `ClientProtocolPipeline` interface. This will allow us to keep
  connection logic for core separated from protocol logic.
* implement pipelines for Cache and Locator.
* some other small fixes and changes.

Signed-off-by: Galen O'Sullivan <go...@pivotal.io>

This closes #873

Signed-off-by: Brian Rowe <br...@pivotal.io>


> Refactor: ClientProtocolService, ClientStreamProcessor
> ------------------------------------------------------
>
>                 Key: GEODE-3751
>                 URL: https://issues.apache.org/jira/browse/GEODE-3751
>             Project: Geode
>          Issue Type: Improvement
>          Components: client/server
>            Reporter: Galen O'Sullivan
>            Assignee: Galen O'Sullivan
>
> The protobuf protocol logic is currently partly in {{GenericProtocolServerConnection}}, but authentication shouldn't really be assumed if it's a generic connection. Refactoring to a single service, with a pipeline that holds handshake/auth/message processing state, will allow us to improve authentication/authorization as well as add a handshake with much less headache.
> This is blocking GEODE-3705 and GEODE-3739.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)