[GitHub] [incubator-pegasus] acelyc111 edited a comment on issue #884: add ‘BATCH_GET’ rpc request for read optimization

[GitBox <gi...@apache.org>]

acelyc111 edited a comment on issue #884:
URL: https://github.com/apache/incubator-pegasus/issues/884#issuecomment-1019689019


   checkAndMutate is a SET type interface in all cases, it is different from batchGet/Set/Del interfaces.
   1. The responses are different, for SET/DEL type interfaces, there are no values responed, but for GET type interfaces, you should design a thrfit struct to contain values, and the related hashkeys and sortkeys along.
   2. Mix all type of requests in one RPC will make code coupled. Imagine the pains we suffered from `multi_get_request`.
   
   So I support to distinguish these RPCs, and try to avoid overdesign.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org