mapleFU opened a new pull request, #1839:
URL: https://github.com/apache/kvrocks/pull/1839

   After review https://github.com/apache/kvrocks/pull/1798 , the patch is 
great, here is just some style enhancement:
   1. `RDBStream::Read` returns `StatusOr<size_t>`, but if status is ok, the 
size is always equal to input, so change it to return `Status`
   2.  Some `move` rather than copy


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to