Re: json(b)_to_tsvector with numeric values - Mailing list pgsql-hackers

From Dmitry Dolgov
Subject Re: json(b)_to_tsvector with numeric values
Date
Msg-id CA+q6zcW5p8i7+0Lv01EUZBaih_nRk_bu91SZM_NHWwtVhwhTeg@mail.gmail.com
Whole thread Raw
In response to Re: json(b)_to_tsvector with numeric values  (Teodor Sigaev <teodor@sigaev.ru>)
Responses Re: json(b)_to_tsvector with numeric values  (Teodor Sigaev <teodor@sigaev.ru>)
List pgsql-hackers
> On 7 April 2018 at 17:09, Teodor Sigaev <teodor@sigaev.ru> wrote:
>>> See workable sketch for parsing jsonb flags and new worker variant.
>>
>>
>> Yep, thanks for the sketch. Here is the new version of patch, does it look
>> close to what you have in mind?
>
>
> Patch looks good except error messaging, you took it directly from sketch
> where I didn't spend time for it. Please, improve. elog() should be used
> only for impossible error, whereas user input could contins mistakes.

I assume what you mean is that for user input errors we need to use ereport.
Indeed, thanks for noticing. I've replaced all elog except the last one, since
it actually describes an impossible situation, when we started to read an
array, but ended up having something else instead WJB_END_ARRAY.

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] [PATCH] Incremental sort
Next
From: Andres Freund
Date:
Subject: Re: [HACKERS] [PATCH] Incremental sort