Re: jsonpath

Hi!

Thank you for your review! Let me answer some points of your review
while others will be answered later.

On Wed, Jan 30, 2019 at 5:28 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2019-01-29 04:00:19 +0300, Alexander Korotkov wrote:
> > +/*****************************INPUT/OUTPUT*********************************/
>
> Why do we need this much code to serialize data that we initially got in
> serialized manner? Couldn't we just keep the original around?

As I get, you concern related to fact that we have jsonpath in text
form (serialized) and convert it to the binary form (also serialized).
Yes, we should do so. Otherwise, we would have to parse jsonpath for
each evaluation. Basically, for the same reason we have binary
representation for jsonb.

> > +++ b/src/backend/utils/adt/jsonpath_exec.c
>
> Gotta say, I'm unenthusiastic about yet another execution engine in some
> PG subsystem.

Better ideas? I can imagine we can evade introduction of jsonpath
datatype and turn jsonpath into executor nodes. But I hardly can
imagine you would be more enthusiastic about that :)

> > @@ -206,6 +206,22 @@ Section: Class 22 - Data Exception
> > 2200N E ERRCODE_INVALID_XML_CONTENT invalid_xml_content
> > 2200S E ERRCODE_INVALID_XML_COMMENT invalid_xml_comment
> > 2200T E ERRCODE_INVALID_XML_PROCESSING_INSTRUCTION invalid_xml_processing_instruction
> > +22030 E ERRCODE_DUPLICATE_JSON_OBJECT_KEY_VALUE duplicate_json_object_key_value
> > +22031 E ERRCODE_INVALID_ARGUMENT_FOR_JSON_DATETIME_FUNCTION invalid_argument_for_json_datetime_function
> > +22032 E ERRCODE_INVALID_JSON_TEXT invalid_json_text
> > +22033 E ERRCODE_INVALID_JSON_SUBSCRIPT invalid_json_subscript
> > +22034 E ERRCODE_MORE_THAN_ONE_JSON_ITEM more_than_one_json_item
> > +22035 E ERRCODE_NO_JSON_ITEM no_json_item
> > +22036 E ERRCODE_NON_NUMERIC_JSON_ITEM non_numeric_json_item
> > +22037 E ERRCODE_NON_UNIQUE_KEYS_IN_JSON_OBJECT non_unique_keys_in_json_object
> > +22038 E ERRCODE_SINGLETON_JSON_ITEM_REQUIRED singleton_json_item_required
> > +22039 E ERRCODE_JSON_ARRAY_NOT_FOUND json_array_not_found
> > +2203A E ERRCODE_JSON_MEMBER_NOT_FOUND json_member_not_found
> > +2203B E ERRCODE_JSON_NUMBER_NOT_FOUND json_number_not_found
> > +2203C E ERRCODE_JSON_OBJECT_NOT_FOUND object_not_found
> > +2203F E ERRCODE_JSON_SCALAR_REQUIRED json_scalar_required
> > +2203D E ERRCODE_TOO_MANY_JSON_ARRAY_ELEMENTS too_many_json_array_elements
> > +2203E E ERRCODE_TOO_MANY_JSON_OBJECT_MEMBERS
> > too_many_json_object_members
>
> Are all of these invented by us?

None of them are invented by us. All of them are part of SQL 2016 standard.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company