JasperFx / weasel

Database Development Made Easy for .Net
MIT License
65 stars 19 forks source link

Weasel.Postgresql.Tables.WriteDropIndex is hardcoded to write 'drop index concurrently' which is very problematic and ignores IndexDefinition.IsConcurrent #29

Closed rwasef1830 closed 2 years ago

rwasef1830 commented 2 years ago

Hello, "drop index concurrently" is not safe to use because Marten might be triggered to run after other DDL or in another transaction. It's generally problematic and should be avoided, or at least respect IndexDefinition.IsConcurrent