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

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2022/01/06 18:00:47 UTC

[GitHub] [zookeeper] pfcoperez commented on pull request #1785: ZOOKEEPER-4425: `snap` 4lw command enabling on demand snapshots

pfcoperez commented on pull request #1785:
URL: https://github.com/apache/zookeeper/pull/1785#issuecomment-1006798539


   Thanks a lot @symat ! I will do as requested. 
   
   I have a branch including exponential back-off for the HTTP version of the command and in fact that logic can be abstracted out as a decorator around any other command that might need it. Maybe you folks would be interested on that as a follow-up contribution too.
   
   > the new command will not be accessible through the 4-letter-words interface, unless it is explicitly whitelisted
   
   That's intentional from my part I think it is a good idea to have it that way.


-- 
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: notifications-unsubscribe@zookeeper.apache.org

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