[Tarantool-patches] [PATCH 2/2] sql: clarify "sql_defer_foreign_keys" setting name
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Sun Jul 12 18:31:58 MSK 2020
Hi! Thanks for the patch!
See 3 comments below.
On 08.07.2020 13:47, Roman Khabibov wrote:
> Raname "sql_defer_foreign_keys" setting to
> "sql_defer_foreign_key_changes".
1. 'Raname' -> 'Rename'.
2. I think the new name is worse. Changes are done immediately.
You can't defer them. The flag means defer the foreign key checks,
not changes. To allow temporary inconsistency if a transaction needs
it, but still check FKs when it will commit.
> Follow up #4511
>
> @TarantoolBot document
> Title: sql_defer_foreign_keys in _space_settings
3. There is no such space '_space_settings'.
> Rename "sql_defer_foreign_keys" setting to
> "sql_defer_foreign_key_changes".
More information about the Tarantool-patches
mailing list