Skip to content

[uc] Improve chunk upload AssemblyStream performance#1370

Merged
MorrisJobke merged 1 commit into
masterfrom
dav-fixassemblystreamperf
Sep 13, 2016
Merged

[uc] Improve chunk upload AssemblyStream performance#1370
MorrisJobke merged 1 commit into
masterfrom
dav-fixassemblystreamperf

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Downstreaming of owncloud/core#26062

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Sep 12, 2016
@icewind1991 icewind1991 added this to the Nextcloud 11.0 milestone Sep 12, 2016
@mention-bot

Copy link
Copy Markdown

@icewind1991, thanks for your PR! By analyzing the annotation information on this pull request, we identified @DeepDiver1975, @LukasReschke and @nickvergessen to be potential reviewers

@nickvergessen

Copy link
Copy Markdown
Member

Let me smash this, to be sure

@nickvergessen

Copy link
Copy Markdown
Member

Ran the tests for big files, they still pass

@MorrisJobke

Copy link
Copy Markdown
Member

I pushed the updated autoloader code. cc @rullzer

@MorrisJobke

Copy link
Copy Markdown
Member

👍

@MorrisJobke

Copy link
Copy Markdown
Member

I pushed the updated autoloader code. cc @rullzer

Ah ... @icewind1991 just pushed shortly before me 😄

@MorrisJobke MorrisJobke force-pushed the dav-fixassemblystreamperf branch from 69ac1ef to d404424 Compare September 12, 2016 14:47
@MorrisJobke

Copy link
Copy Markdown
Member

Ah ... @icewind1991 just pushed shortly before me 😄

Rebased to resolve the conflict

@rullzer rullzer force-pushed the dav-fixassemblystreamperf branch from d404424 to 1ab472b Compare September 13, 2016 07:29
@rullzer

rullzer commented Sep 13, 2016

Copy link
Copy Markdown
Member

LGTM

@MorrisJobke MorrisJobke merged commit 15e7f8f into master Sep 13, 2016
@MorrisJobke MorrisJobke deleted the dav-fixassemblystreamperf branch September 13, 2016 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants