RamiAwar / dataline

Chat with your data - AI data analysis and visualization on CSV, Postgres, MySQL, Snowflake, SQLite...
https://dataline.app
GNU General Public License v3.0
129 stars 6 forks source link

Fix/streaming endpoint sqlite lock #193

Closed anthony2261 closed 3 weeks ago

anthony2261 commented 3 weeks ago

Issue #, if available:

190

Description of changes: There's a breaking change in the newer versions of fastapi changing the behavior Depends w.r.t. background tasks and streaming responses. The session was being committed and closed before the query was able to run.

Changes: used a get_session_no_commit depends so that we manually commit and close the session after the query runs. Also brought back Docker as it was innocent this entire time.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.