[GitHub] [incubator-kvrocks] ShooterIT commented on issue #1350: Add the support of the blocking migration for the cluster migrate command

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

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

   I prefer to use SYNC|ASYNC flag instead of another command which has similar arguments check and logic.
   in new redis version, they don't want to add more commands, such as flush command, it has ASYNC option.
   
   For `lpop` and `blpop`, they operate data space, different commands have different flags so redis can distinguish which command can execute in scripts env  easily.


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