[GitHub] [zookeeper] narendly commented on pull request #1492: ZOOKEEPER-3419: Backup and recovery support.

[GitBox <gi...@apache.org>]

narendly commented on pull request #1492:
URL: https://github.com/apache/zookeeper/pull/1492#issuecomment-729188503


   @hanm 
   Thanks for the follow-up! This is helpful. Just so I don't leave you in the dark, I am actively reviewing this. There are a few things that I think would also be beneficial (both to the functionality and to make it more compatible to be integrated into open source):
   
   1. Generifying components such as the storage provider implementation in QuorumPeerMain.java and BackupConfig. I do see that an effort was made throughout the code, but perhaps they don't make it completely pluggable. We are considering adding support for NFS-based storage as well. 
   2. Adding support for interface that will allow timestamp-Zxid translation. Ideally, we want to have some variant of "point-in-time recovery" with some degree of automation. I think it will jibe well with the automatic backup feature. Will follow up on this in the near future.
   3. (Minor) Hoping to offer some alternatives to the components proprietary to Twitter.
   
   Hope to continue the discussion here or elsewhere (whichever medium is preferred for ZK development - should we move this discussion to ZooKeeper Atlassian wiki/JIRA?). I'm new to the community but eager to contribute. Keep me posted.


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

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