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

Streaming response for CSV import #6

Open
fidel-outdoorsy opened this issue Jul 20, 2019 · 0 comments
Open

Streaming response for CSV import #6

fidel-outdoorsy opened this issue Jul 20, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@fidel-outdoorsy
Copy link
Contributor

Importing can take a long time, and it's common to cause response timeouts on even middle-sized CSV files (dozens of rows).

A good solution is to use a StreamingHTTPResponse object instead of a regular HTTPResponse (like proposed for #3). In this way the server processes the import one instance at a time, sending to the client the import result (success or error) in real time, thus avoiding the timeout and as a side-effect enhancing user experience.

On the frontend side consider using something like OboeJS to parse the streaming JSON that the server will send with the successes or errors of each row.

@fidel-outdoorsy fidel-outdoorsy added the enhancement New feature or request label Jul 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

1 participant