[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