hankinsoft / SQLPro

SQLPro bug & features tracking.
100 stars 26 forks source link

Renaming or dropping a column in a table locks up SQLPro for SQLite #961

Closed GaryMeyers1 closed 4 months ago

GaryMeyers1 commented 4 months ago

This is probably going to be hard for you to reproduce but I have run PRAGMA integrity_check on my database. I tried to rename a column in a table with just 195 rows and 52 columns but SQLPro locked up. I exported the data from the table into a JSON file and then truncated the table, but the rename attempt still locked up SQLPro. I then clicked the minus sign in the upper right corner to drop the column, with the table still being empty, and SQLPro locked up. I downloaded another app from the internet to drop the column from the table. I then launched SQLPro and loaded my JSON data, mapping the deleted column to SKIP.

Expected behavior SQLPro should have renamed the column without an error. SQLPro should have dropped the column without an error.

Screenshots

column rename column drop

Environment details (please complete the following information):

GaryMeyers1 commented 4 months ago

I have old versions of my database that I can delete records from, to make them small enough to email to Kyle, after confirming that the rename and drop operations also fail on those smaller files.

GaryMeyers1 commented 4 months ago

I emailed a sample database to Kyle that locks up when dropping a column.

hankinsoft commented 4 months ago

Reproduced and am working on a fix now.

hankinsoft commented 4 months ago

Could you try this build:

https://sqlprostudio.s3.us-east-1.amazonaws.com/sqlite/SQLProSQLite.2024.12.app.zip

And let me know if that helps out?

GaryMeyers1 commented 4 months ago

I both dropped and renamed a column in this table from a mid-February version of my database. Both operations succeeded without locking up SQLPro.

Thank you, Gary

On Mar 4, 2024, at 12:26 PM, Kyle Hankinson @.***> wrote:

Could you try this build:

https://sqlprostudio.s3.us-east-1.amazonaws.com/sqlite/SQLProSQLite.2024.12.app.zip

And let me know if that helps out?

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

hankinsoft commented 4 months ago

Great, thanks for letting me know!