You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/12/24 19:39:22 UTC

[GitHub] [incubator-kvrocks] torwig opened a new pull request, #1205: Refactor return value of functions that related to a slot migration

torwig opened a new pull request, #1205:
URL: https://github.com/apache/incubator-kvrocks/pull/1205

   Make functions return the value of type `Status` instead of `bool`.
   Log errors on a higher level by passing error messages to the caller via `Status`.
   Tidy the constructor of the `SlotMigrate` class.
   
   @PragmaTwice The `.Prefixed` method of the `Status` type helped me a lot to make this refactoring. Thank you :)


-- 
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: issues-unsubscribe@kvrocks.apache.org

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

[GitHub] [incubator-kvrocks] git-hulk merged pull request #1205: Refactor return value of functions that related to a slot migration

Posted by GitBox <gi...@apache.org>.

git-hulk merged PR #1205:
URL: https://github.com/apache/incubator-kvrocks/pull/1205


-- 
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: issues-unsubscribe@kvrocks.apache.org

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