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

DS-2359 Error when depositing large files via browser (over 2Gb) #1723

Merged
merged 1 commit into from Jul 7, 2017

Conversation

PhilipVis
Copy link
Contributor

Implemented the change proposed by Bram in jira ticket:
https://jira.duraspace.org/browse/DS-2359

I created a new DSpacePartOnDisk class based on the cocoon PartOnDisk class. This class contains the fix to replace the nast that was causing problems with file uploads that are larger than 2GB.
Update classes that used the PartOnDisk class to use the DSpacePartOnDisk class instead.

@mwoodiupui mwoodiupui changed the title DS-2359 DS-2359 Error when depositing large files via browser (over 2Gb) Apr 20, 2017
@tdonohue tdonohue added bug quick win Pull request is small in size & should be easy to review and/or merge labels Apr 20, 2017
@tdonohue tdonohue added this to the 6.1 milestone Apr 20, 2017
@terrywbrady
Copy link
Contributor

@PhilipVis , @bram-atmire , I am interested in testing this for inclusion in 6.1.

As a baseline, I attempted to upload a 2.4G file via XMLUI (without this patch). The upload gets to 100% and there is no indication of success or failure. Do I need something on the UI side to process a large upload?

@terrywbrady terrywbrady self-requested a review June 27, 2017 19:23
Copy link
Contributor

@terrywbrady terrywbrady left a comment

Choose a reason for hiding this comment

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

+1, I tested locally with a 2.4G file. The process fails without the patch and succeeds when it is present. That is the extent of my testing.

@tdonohue
Copy link
Member

tdonohue commented Jul 7, 2017

👍 Code looks good, and this has received another test/review. This is also standalone, so I'm going to merge this for 6.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug quick win Pull request is small in size & should be easy to review and/or merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants