so 2. 3. 2019 v 6:15 odesÃlatel Alexander Korotkov < a.korot...@postgrespro.ru> napsal:
> Hi! > > On Fri, Mar 1, 2019 at 3:36 AM Nikita Glukhov <n.glu...@postgrespro.ru> > wrote: > > > > Attached 34th version of the patches. > > > > 1. Partial jsonpath support: > > - Fixed copying of jsonb with vars jsonb_path_query() into SRF context > > - Fixed error message for jsonpath vars > > - Fixed file-level comment in jsonpath.c > > > > 2. Suppression of numeric errors: > > Now error handling is done without PG_TRY/PG_CATCH using a bunch of > internal > > numeric functions with 'bool *error' flag. > > Revised patches 1 and 2 are attached. Changes are following > > * Small refactoring, comments adjustment and function renaming. In > particular, I've removed "recursive" prefix from function names, > because it actually not that informative assuming header comment > explains that the whole jsonpath executor is recursive. Also, I made > "Unwrap" suffix more clear. Not it's distinguished what is unwrapped > target (UnwrapTarget) or result (UnwrapResult). Also, now it's clear > that function doesn't always unwraps but has an option to do so > (OptUnwrap). > * Some more cases are covered by regression tests. > These patches are large, but I think so the granularity and modularity of these patches are correct. Now, it looks very well. Pavel > ------ > Alexander Korotkov > Postgres Professional: http://www.postgrespro.com > The Russian Postgres Company >
