[GitHub] [incubator-pegasus] acelyc111 commented on pull request #728: improve performance of count_data

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

acelyc111 commented on PR #728:
URL: https://github.com/apache/incubator-pegasus/pull/728#issuecomment-1193748476

   共用rpc我觉得倒没有问题。问题点应该在count数值应该放在哪里，序列化在 `struct get_scanner_request` 里面不是很合适，在 `scan_request` 里面加一个标记表示是否只取count，并在 `scan_response` 把count结果带出来或许更合适。
   新增的字段都是 optional 的，当server端是老版本，即不支持这个新增字段的判断时，会使用老接口逻辑把kv原始数据带出去，新的client可以以此判断server端是否支持，如果不支持，业务逻辑层可以返回失败。
   更干净的做法当然还是新增一个独立的rpc，但是get_scanner还是可以复用的。


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