[tarantool-patches] Re: [PATCH v4 5/6] sql: move Triggers to server

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Sun Jun 24 23:41:11 MSK 2018


>> @@ -3252,6 +3303,88 @@ on_replace_dd_trigger(struct trigger * /* trigger */, void *event)
>> {
>> +		const char *space_opts =
>> +			tuple_field_with_type_xc(new_tuple,
>> +						 BOX_TRIGGER_FIELD_OPTS,
>> +						 MP_MAP);
> 
> Why do we still hold string of ‘CREATE TRIGGER’ statement as a map?
> AFAIU trigger can’t feature any functional elements except for mentioned one.

In SQL - can not, but when we will persist Lua trigger, the map contains: body,
after/before/instead, delete/insert/replace etc. So lets leave it be map.




More information about the Tarantool-patches mailing list