-
Notifications
You must be signed in to change notification settings - Fork 495
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
File Upload: completed files remain in the "select files to add" box after finishing upload #2880
Comments
I'll probably try to reproduce this with files from https://data.sbgrid.org/dataset/133/ based on what @pameyer mentioned at http://irclog.iq.harvard.edu/dataverse/2016-01-20 |
For reproducibility, I was seeing this on 4.2.2 |
Thanks. I wonder if the upgrade from Primefaces 4 to 5 in #2730 as part of 4.2.3 will make any difference in reproducing this bug. |
On my laptop I'm on commit a90be2a of the 4.2.3 branch (which already has the Primefaces 5 upgrade added in #2730) and I just tried uploading all 148 files from https://data.sbgrid.org/dataset/133/ The main thing I want to do for now is upload a screenshot of what I'm seeing, which I'll put below, to make sure this is what @pameyer is talking about. Rather than 2-5 files showing up under "Select Files to Add" there are 8 in the screenshot. Other observations (while using Firefox) are that my Mac was "beachballing" during the upload and I couldn't switch tabs. I'm pretty sure my fans were blowing pretty hard. That is to say this feel like an expensive operation for a browser. Update: I showed the screenshot to @scolapasta and we agree that it's a bug that dos_mad_2_070.img, for example, shows at the top under "Select Files to Add" since dos_mad_2_070.img appears in the file list lower down. |
@pdurbin - that screenshot matches what I was seeing. The number of files varied (I deleted and re-uploaded a few times). I didn't notice any beachballing - but I wasn't watching for it. |
@pdurbin - the fans spinning up on your laptop might have been the dataverse app (ingestion?) rather than the browser? |
@pameyer I had that thought as well but I don't have any evidence one way or the other. :) |
@bmckinney for now can you please see if you can reproduce this in the "develop" branch? |
@pdurbin and @pameyer - confirmed with develop branch Also, not only do random successful uploads linger in the "Select files" section, but one of the 120 files I attempted to upload failed to upload and it wasn't listed in either section. If I count the files in dataverse's temp folder (prior to saving the dataset), it is also 1 shy of the 120 total. This bug is more concerning to me! |
I assume you're talking about the JSF temp folder? (#2848 is about documenting all of them.) If so, you could repeat your test via SWORD to avoid JSF. |
also seeing lots of java.util.ConcurrentModificationException when checking for duplicates |
yeah, i think edu.harvard.iq.dataverse.EditDatafilesPage#isDuplicate (which generates java.util.ConcurrentModificationException) is the culprit. when i comment out the md5 stuff in that method and return false in all cases, i'm unable to reproduce the bug (all files reported uploaded and no lingering files in the top section) |
There is a pull request made to fix this. Will test and close. |
Tested the above scenario with the mentioned files. Was able to upload twice without errors or missing/ hung files. Closing this particular issue but as mentioned, some weirdness remains and will be addressed in #3116. Closing. |
Uploaded ~150 ~11 MB files individually (not in zip/tgz)
Possibly related : #2491 , #2482
The text was updated successfully, but these errors were encountered: