[stable9] Chunk upload for GDrive #23361

Merged
merged 2 commits into from Mar 22, 2016

Projects

None yet

5 participants

@PVince81
Collaborator

Backport of #23323 to stable9

Please review @icewind1991 @LukasReschke @schiesbn @Xenopathic @MorrisJobke

PVince81 added some commits Mar 16, 2016
@PVince81 PVince81 Chunk upload for GDrive
Instead of storing the WHOLE file in memory in a PHP variable, use the
library's chunk upload support.
8e0b78c
@PVince81 PVince81 Only use GDrive chunks when needed
1e1c088
@PVince81 PVince81 added this to the 9.0.1-current-maintenance milestone Mar 17, 2016
@mention-bot

By analyzing the blame information on this pull request, we identified @Xenopathic, @MTGap and @icewind1991 to be potential reviewers

@PVince81 PVince81 referenced this pull request Mar 17, 2016
Merged

Chunk upload for GDrive #23323

5 of 5 tasks complete
@PVince81
Collaborator

@owncloud/qa can you help testing ?

@SergioBertolinSG
Member

Works fine 👍

A problem with an upload happened once, but not reproducible. Logs here http://paste.debian.net/417740/

@PVince81
Collaborator

Looks like the error you saw was a connection failed to the GDrive server in the constructor.

@PVince81
Collaborator

@icewind1991 second review ?

@icewind1991
Member

👍

@DeepDiver1975 DeepDiver1975 merged commit a9c13dc into stable9 Mar 22, 2016

22 checks passed

cla-bot-core Build #2394 succeeded in 17 sec
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
core-ci-linux-jsunit/database=sqlite,label=SLAVE Build #59607 succeeded in 35 sec
Details
core-ci-linux-swift-primary-storage/database=mysql,label=SLAVE Build #53555 succeeded in 9 min 14 sec
Details
core-ci-linux/database=mysql,label=SLAVE Build #28189 succeeded in 23 min
Details
core-ci-linux/database=oci,label=SLAVE Build #28189 succeeded in 42 min
Details
core-ci-linux/database=pgsql,label=SLAVE Build #28189 succeeded in 22 min
Details
core-ci-linux/database=sqlite,label=SLAVE Build #28189 succeeded in 10 min
Details
ocs-api-integration-tests-ci Build #8566 succeeded in 8 min 45 sec
Details
server-master-linux-externals-ci/database=sqlite,external=smb-silvershell,label=SLAVE Build #8380 succeeded in 58 sec
Details
server-master-linux-externals-ci/database=sqlite,external=swift-ceph,label=SLAVE Build #8380 succeeded in 2 min 52 sec
Details
server-master-linux-externals-ci/database=sqlite,external=webdav-ownCloud,label=SLAVE Build #8380 succeeded in 3 min 2 sec
Details
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #8811 succeeded in 4 min 25 sec
Details
server-master-linux-php5.4-ci/database=sqlite,label=SLAVE Build #951 succeeded in 3 min 58 sec
Details
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #36404 succeeded in 2 min 46 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=litmus,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 8 min 28 sec
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@0,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 59 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@1,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 56 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_shareLink,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 26 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePermissions,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 52 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationGroups,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 16 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationInsideGroups,mirallBranch=v2.0.2,slave=SMASH Build #12759 succeeded in 13 min
Details
@DeepDiver1975 DeepDiver1975 deleted the stable9-gdrive-chunkupload branch Mar 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment