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

Configuration for Availability of "Download All Dataset" #7714

Closed
djbrooke opened this issue Mar 23, 2021 · 2 comments · Fixed by #7806
Closed

Configuration for Availability of "Download All Dataset" #7714

djbrooke opened this issue Mar 23, 2021 · 2 comments · Fixed by #7806
Assignees

Comments

@djbrooke
Copy link
Contributor

djbrooke commented Mar 23, 2021

The "Download All Dataset" feature introduced to the Dataverse Software in 5.0 has been working great on the Harvard Dataverse Repository for the last several months. :) However, we should consider that:

  • On the Harvard Dataverse Repository we in parallel activated the zipper external service to move the zipping off the application server, which means we haven't run into system stability issues. Not every installation will enable this service.
  • Because of improvements in large data uploads, we're now seeing some datasets that are pretty big. See Ryan Enos’ recent dataset at 100+ GB (https://dataverse.harvard.edu/dataset.xhtml?persistentId=doi:10.7910/DVN/A40X5L). Providing a good user experience around 100+ GB Zip downloads through the UI may be difficult, and there may be cost considerations for some installations.

A proposal surfaced in #7557 is to use the :ZipDownloadLimit setting to control whether or not the Download All option is available. I think this could serve the two use cases of:

  • As a Dataverse installation administrator I want to provide download options that are more suited for my infrastructure/cost
  • As a Dataverse installation downloader I want to only be presented with a "Download All" option when I'm going to get a zip of all of the files

I think we should move forward with this.

Three things to note:

  • The UI impact here would be to either not have the button, or provide the button and some messaging about why the zip cannot be downloaded. To discuss.
  • after implementing this, we'll still have the issue of incomplete zips (with manifest.txt showing the missing files) when someone goes through and selects a number of files that are above the :ZipDownloadLimit, but it won't happen with the prominently featured "Download All" option at the top of the page. I don't want to solve this right now.
  • that the :ZipDownloadLimit is ignored when the zipper service is available, so in those installations running the zipper service this would solely be a setting that controls the availability of Download All. I considered suggesting a new option to be more specific, but I'll let more technical people discuss the best approach.
@djbrooke djbrooke changed the title New Configuration for Availability of "Download All Dataset" Configuration for Availability of "Download All Dataset" Mar 23, 2021
@djbrooke
Copy link
Contributor Author

  • the option to show an informative message about why the option is not available is more desirable than not showing the button (we should figure this out first)
  • we have the dataset size easily available, so from a code perspective this should be easy
  • we should apply these changes to the API as well - an error code and a message. Note that the crawlable access can provide this functionality for API users.

@sekmiller sekmiller self-assigned this Mar 26, 2021
sekmiller added a commit that referenced this issue Apr 1, 2021
sekmiller added a commit that referenced this issue Apr 5, 2021
sekmiller added a commit that referenced this issue Apr 8, 2021
sekmiller added a commit that referenced this issue Apr 9, 2021
sekmiller added a commit that referenced this issue Apr 9, 2021
sekmiller added a commit that referenced this issue Apr 12, 2021
sekmiller added a commit that referenced this issue Apr 14, 2021
sekmiller added a commit that referenced this issue Apr 15, 2021
sekmiller added a commit that referenced this issue Apr 16, 2021
sekmiller added a commit that referenced this issue Apr 23, 2021
sekmiller added a commit that referenced this issue Apr 26, 2021
sekmiller added a commit that referenced this issue May 3, 2021
@TaniaSchlatter
Copy link
Member

Use cases and options for messaging in the Access dropdown and in a popup. See the Design doc
Screen Shot 2021-06-08 at 1 07 20 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants