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

Galen O'Sullivan updated GEODE-3386:
------------------------------------
    Description: 
For logical separation of the new client API, it will be better to have an 
Error that is contained by ErrorResponse, rather than having KeyedErrorResponse 
contain an ErrorResponse.

In pseudo-protobuf,
{code}
PutAllResponse {
  repeated Entry successes = 1,
  repeated KeyedErrorResponse errors = 2,
}
KeyedErrorResponse {
  Key,
  ErrorResponse
}
ErrorResponse {
  string
}
{code}
instead,
{code}
KeyedErrorResponse {
  Key,
  Error,
}
{code}
and 
{code}
ErrorResponse {
Error
}

Error {
string
}
{code}

  was:
For logical separation of the new client API, it will be better to have an 
Error that is contained by ErrorResponse, rather than having KeyedErrorResponse 
contain an ErrorResponse.

{code}
PutAllResponse {
  repeated Entry successes = 1,
  repeated KeyedErrorResponse errors = 2,
}
KeyedErrorResponse {
  Key,
  ErrorResponse
}
ErrorResponse {
  string
}
{code}
instead,
{code}
KeyedErrorResponse {
  Key,
  Error,
}
{code}
and 
{code}
ErrorResponse {
Error
}

Error {
string
}
{code}


> Create Error type for KeyedErrorResponse and ErrorResponse
> ----------------------------------------------------------
>
>                 Key: GEODE-3386
>                 URL: https://issues.apache.org/jira/browse/GEODE-3386
>             Project: Geode
>          Issue Type: Sub-task
>          Components: client/server
>            Reporter: Galen O'Sullivan
>
> For logical separation of the new client API, it will be better to have an 
> Error that is contained by ErrorResponse, rather than having 
> KeyedErrorResponse contain an ErrorResponse.
> In pseudo-protobuf,
> {code}
> PutAllResponse {
>   repeated Entry successes = 1,
>   repeated KeyedErrorResponse errors = 2,
> }
> KeyedErrorResponse {
>   Key,
>   ErrorResponse
> }
> ErrorResponse {
>   string
> }
> {code}
> instead,
> {code}
> KeyedErrorResponse {
>   Key,
>   Error,
> }
> {code}
> and 
> {code}
> ErrorResponse {
> Error
> }
> Error {
> string
> }
> {code}



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

Reply via email to