hankinsoft / SQLPro

SQLPro bug & features tracking.
106 stars 28 forks source link

Crashes during or following import #982

Closed gitjoost closed 5 months ago

gitjoost commented 5 months ago

crashreports.zip Describe the bug App crashes during CSV import (approx 500 lines, 7 columns) or crashes a short while thereafter.

To Reproduce Import any csv after having had the app open and worked with for "a while". "a while" appears to be 5-10 minutes or longer. Import is successful (always) when doing the import after a restart (suggesting csv format is not to blame). It appears that when working with the app for a while (queries, data review etc and possibly an import) an import (of any format) crashes the app. memory leak that builds up?

"appears" to happen a bit more frequently as of late? I really don't know because I did not keep stats, it just "feels" this is happening more often then before.

Expected behavior No crash

Screenshots Crash report attached

Environment details (please complete the following information):

Additional context I have a crash reports attached

gitjoost commented 5 months ago

Forgot to mention; csv imports are extremely slow, a 1000 lines, 8 columns, easily take 5 minutes or more. Compared to sqlite3 command line or other sqlite tools which take a second or so. Not sure if this has any bearing on the problem reported above but it sure affects productivity.

hankinsoft commented 5 months ago

Would it be possible for you to provide a CSC file that replicates this issue? Along with the ddl for a target table? Email would be fine as well.

gitjoost commented 5 months ago

Hi Kyle, the specific csv contains a lot of financial info I cannot make available but this issue occurs regardless of the specific data. I am attaching a CSV with a portion of a pedigree database which has caused the same issue for me in the past.

On May 26, 2024, at 10:16 AM, Kyle Hankinson @.***> wrote:

Would it be possible for you to provide a CSC file that replicates this issue? Along with the ddl for a target table? Email would be fine as well.

— Reply to this email directly, view it on GitHub https://github.com/hankinsoft/SQLPro/issues/982#issuecomment-2132290486, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAPC2MMOZSZIVYRYLCPDBADZEIKIBAVCNFSM6AAAAABIH4GZPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGI4TANBYGY. You are receiving this because you authored the thread.

hankinsoft commented 5 months ago

Hi - would it be possible to post the CSV from the GitHub website? Unfortunately, attachments via email do not get uploaded.

hankinsoft commented 5 months ago

I think I should be ok without it. After some more testing, I believe I have this reproduced. Investigating it now.

hankinsoft commented 5 months ago

If you get the chance, can you try https://sqlprostudio.s3.us-east-1.amazonaws.com/sqlite/SQLProSQLite.2024.30.app.zip? It should help with both this and #983.

gitjoost commented 5 months ago

Speed has been substantially improved (!) and so far no crashes, nice! I am going to give it some more time but it looks like you might close this issue, thanks for your support.

hankinsoft commented 5 months ago

Glad to hear that seems to have sorted it.