[GitHub] [incubator-kvrocks] ellutionist commented on issue #1350: Add blocking clusterx subcommand "bmigrate"

["ellutionist (via GitHub)" <gi...@apache.org>]

ellutionist commented on issue #1350:
URL: https://github.com/apache/incubator-kvrocks/issues/1350#issuecomment-1483980049

   Thanks to @git-hulk  and @xiaobiaozhao . This is a good workaround to not create a new subcommand and to provide backward compatibility. 
   
   However, I have to mention this: the "OK" returned by `migrate` without `timeout` argument means "migration started". It differs from the "OK" returned by `mirate` with `timeout` which means "migration finished". Such inconsitency may cause confusion to our users, especially who is new to Kvrocks.
   
   If this is acceptable, then I am ready to implement it.


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