[GitHub] geode issue #649: GEODE-2997: Change new protocol GetAllResponse.

[galen-pivotal <gi...@git.apache.org>]

Github user galen-pivotal commented on the issue:

    https://github.com/apache/geode/pull/649
  
     Some other thoughts from today:
    * make GetAllResponse return a collection of Entries and a collection of errors.
    * make Get and GetAll return the same type of response, as mentioned above. Get will always return a single key in either the success or error collections.
       - maybe looking in a collection is extra overhead for a simple Get request?
       - bigger errors (like malformed messages) will still return errors at the Response level rather than errors nested in the response message.
    
    Thoughts?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---