Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Complete] Simple replication #223

Merged
merged 13 commits into from
May 15, 2017
Merged

[Complete] Simple replication #223

merged 13 commits into from
May 15, 2017

Conversation

omad
Copy link
Member

@omad omad commented Apr 28, 2017

Add a simple replication tool.

There's masses of room for improvement, but I found this useful for downloading data from the AGDC to my local PC before outages at the NCI.

@mpaget
Copy link
Contributor

mpaget commented May 11, 2017

@omad: could you add some notes to documentation/Components and Interfaces.md > Managed Replica reflecting what this commit does and does not do relative to the intentions of the general solution, please?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.239% when pulling 7ca43ae on simple_replication into 58a8fcf on develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.239% when pulling 02fcdbc on simple_replication into 58a8fcf on develop.

@omad omad requested a review from andrewdhicks May 15, 2017 07:24
@omad omad changed the title Simple replication [Done] Simple replication May 15, 2017
@omad omad changed the title [Done] Simple replication [Complete] Simple replication May 15, 2017
@andrewdhicks
Copy link
Contributor

Very clear and straight-forward code. Good documentation.

It is specific to file-based storage systems, so we may have to modify how it works when we abstract the storage mechanism to a driver-based model.

@mpaget I'll add a link to the simple replication readthedocs page to the components page, as a reference point for the implementer of the managed replica components.

@andrewdhicks andrewdhicks merged commit c490244 into develop May 15, 2017
@omad omad deleted the simple_replication branch May 16, 2017 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants