[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