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

Data import: limit csv file size #834

Closed
dpalomino opened this issue Oct 14, 2016 · 0 comments
Closed

Data import: limit csv file size #834

dpalomino opened this issue Oct 14, 2016 · 0 comments
Assignees
Milestone

Comments

@dpalomino
Copy link

As the import process done via a .csv file is not executed asynchronously right now, we would need to limit the file size of the .csv import file to avoid too long import executions that will be prone to errors (like user trying to reload the page, getting some server timeout errors, etc).

As an estimation, a .csv file containing 1k records with 35 columns is 500KB. Maybe that could be a good threshold.

@oliverroick oliverroick added this to the Sprint 10 milestone Oct 18, 2016
bjohare pushed a commit that referenced this issue Oct 27, 2016
bjohare pushed a commit that referenced this issue Oct 28, 2016
ian-ross pushed a commit that referenced this issue Oct 28, 2016
ian-ross pushed a commit that referenced this issue Oct 28, 2016
@dpalomino dpalomino mentioned this issue Nov 11, 2016
1 task
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

No branches or pull requests

3 participants