shlomi-noach / openarkkit

Automatically exported from code.google.com/p/openarkkit
24 stars 16 forks source link

oak-online-alter-table exiting on error can fail transactions that modify data #33

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The cleanup() method drops the ghost table before dropping the custom triggers. 
For a short moment, this leaves the database in a state where triggers on the 
original table fail along with queries that triggered them because the oak__ 
table doesn't exist anymore. I believe it would be safer to drop the triggers 
first.

Original issue reported on code.google.com by gryw...@gmail.com on 17 Apr 2013 at 12:05

GoogleCodeExporter commented 9 years ago
Yes, this is absolutely correct. I have modified order of rows.
Has been a while since I released a maintenance version; expect to deliver one 
shortly.

Original comment by shlomi.n...@gmail.com on 5 May 2013 at 7:58