[tarantool-patches] Re: [PATCH 1/4] tuple: expose JSON go_to_key and go_to_index functions

Konstantin Osipov kostja.osipov at gmail.com
Mon Sep 16 21:57:57 MSK 2019


* Vladislav Shpilevoy <v.shpilevoy at tarantool.org> [19/09/16 21:40]:

I guess there will be some ceremonial lgtm from someone from
mail.ru team...

> Hi! Thanks for the review!
> 
> On 16/09/2019 09:03, Konstantin Osipov wrote:
> > * Vladislav Shpilevoy <v.shpilevoy at tarantool.org> [19/09/16 09:58]:
> >> They are needed in JSON path update for a case, when a final part
> >> of the path may not exist, and go_to_path returns an error.
> >>
> >> That case is '=' and '!' on not existing fields. For example,
> >> in {'=', '[1][2][3]', 20} field [3] can be not existing.
> >>
> >> For these cases JSON update will have its own implementation of
> >> go_to_path allowing optional last field.
> >>
> >> Needed for #1261
> >> ---
> > 
> > lgtm
> > 
> > 
> 
> Talking of lgtms, I don't know what to do next after your lgtm.
> Who should review and push it now?

-- 
Konstantin Osipov, Moscow, Russia




More information about the Tarantool-patches mailing list