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

2017-08-23 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/649 closing for #739 . --- 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

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

2017-08-15 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/649 @galen-pivotal is this PR active anymore? --- 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

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

2017-07-27 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/649 This isn't code complete; I'm planning to work on this when I have the time (unless anyone else wants to). --- If your project is set up for it, you can reply to this email and have your reply

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

2017-07-20 Thread galen-pivotal
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