Add --delay-updates to sync command #33

Closed
mdomsch opened this Issue Mar 1, 2012 · 4 comments

Comments

Projects
None yet
3 participants
@mdomsch
Member

mdomsch commented Mar 1, 2012

Similar to --delete-after, --delay-updates pushes changed files into place after uploading new content, and before deleting removed content. In this way there is a minimal amount of time when a yum or deb repository is in an inconsistent state on disk.

@ryanlim

This comment has been minimized.

Show comment
Hide comment
@ryanlim

ryanlim Mar 1, 2012

I use my fork of s3cmd to sync my backups to S3. It requires the sync to complete or never at all. Does this my fork/commit work?

ryanlim@7296397

ryanlim commented Mar 1, 2012

I use my fork of s3cmd to sync my backups to S3. It requires the sync to complete or never at all. Does this my fork/commit work?

ryanlim@7296397

@mdomsch

This comment has been minimized.

Show comment
Hide comment
@mdomsch

mdomsch Mar 1, 2012

Member

See pull request #34.

Member

mdomsch commented Mar 1, 2012

See pull request #34.

@mdomsch

This comment has been minimized.

Show comment
Hide comment
@mdomsch

mdomsch Mar 1, 2012

Member

ryanlim: I like your approach too. rsync uses a upload-to-temp-file-and-rename scheme for all of its files. The trick in the combination of the two is to order the (many) renames such that updated files get renamed last. My patch affects the upload order only. Your patch, if it could take into account setting the rename order, would then be the best of both worlds.

Member

mdomsch commented Mar 1, 2012

ryanlim: I like your approach too. rsync uses a upload-to-temp-file-and-rename scheme for all of its files. The trick in the combination of the two is to order the (many) renames such that updated files get renamed last. My patch affects the upload order only. Your patch, if it could take into account setting the rename order, would then be the best of both worlds.

@kevindaub

This comment has been minimized.

Show comment
Hide comment
@kevindaub

kevindaub Mar 5, 2013

Contributor

This was merged with #34, so this can be closed.

Contributor

kevindaub commented Mar 5, 2013

This was merged with #34, so this can be closed.

@mdomsch mdomsch closed this Mar 9, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment