Skip to content

use storage copy implementation when doing dav copy - #24358

Merged
MorrisJobke merged 1 commit into
masterfrom
dav-storage-copy
Dec 3, 2020
Merged

use storage copy implementation when doing dav copy#24358
MorrisJobke merged 1 commit into
masterfrom
dav-storage-copy

Conversation

@icewind1991

Copy link
Copy Markdown
Member

instead of using the sabredav fallback (which does a read+write stream copy)

this greatly speeds up dav copies

Signed-off-by: Robin Appelman robin@icewind.nl

instead of using the sabredav fallback (which does a read+write stream copy)
this greatly speeds up dav copies
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991icewind1991 added the 3. to review Waiting for reviews label Nov 24, 2020
@icewind1991icewind1991 added this to the Nextcloud 21 milestone Nov 24, 2020
@icewind1991icewind1991 mentioned this pull request Nov 24, 2020
@rullzer

Copy link
Copy Markdown
Member

No need to also set it on the File?
I'll try this in a bit.

But I suspect it should speedup copies all around right? As even with local storage you just say to the OS. Copy this there.

@icewind1991

Copy link
Copy Markdown
MemberAuthor

No need to also set it on the File?

sabredav always call the copy for the parent directory of the target from what I can tell

But I suspect it should speedup copies all around right? As even with local storage you just say to the OS. Copy this there.

Should be yes

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy still works 👍

@MorrisJobke
MorrisJobke merged commit fc57f60 into masterDec 3, 2020
@MorrisJobke
MorrisJobke deleted the dav-storage-copy branch December 3, 2020 14:03
@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable20

@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable19

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

Labels

3. to reviewWaiting for reviewsperformance 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@rullzer@MorrisJobke