[tarantool-patches] Re: [PATCH 4/4] schema: explicitly forbid setting sequence for json path key part

[Vladimir Davydov]

On Wed, May 15, 2019 at 04:00:26PM +0300, Konstantin Osipov wrote:
> * Vladimir Davydov <vdavydov.dev at gmail.com> [19/05/15 14:16]:
> > When a space has a sequence, we substitute NULL in the corresponding
> > primary index part with the next value generated by the sequence. We do
> > this by patching raw msgpack, see request_handle_sequence. The problem
> > is we can't do it easily if the field is nested. For example, consider
> > field [1].a.b. In Lua it is impossible to create tuple {{a = {b = nil}}}
> 
> It is trivial to do, use msgpack.NULL

Hmm, I tried box.NULL and it didn't work out.

msgpack.NULL seems to be different.

> 
> > so we would need to restore the whole path from an empty tuple.
> 
> No, it's not the job of the sequence to fill in the entire path.
> You can expect the right prefix to be there.

Okay, will try to address.