On 03.01.24 13:01, Peter Eisentraut wrote:
On 07.12.23 14:24, Jeevan Chalke wrote:
We have the same issue with integer conversion and need a fix.
Unfortunately, I was using int8in() for the conversion of numeric
values. We should be using numeric_int8() instead. However, there is
no opt_error version of the same.
So, I have introduced a numeric_int8_opt_error() version just like we
have one for int4, i.e. numeric_int4_opt_error(), to suppress the
error. These changes are in the 0001 patch. (All other patch numbers
are now increased by 1)
I have used this new function to fix this reported issue and used
numeric_int4_opt_error() for integer conversion.
I have committed the 0001 and 0002 patches for now.
The remaining patches look reasonable to me, but I haven't reviewed them
in detail.
The 0002 patch had to be reverted, because we can't change the order of
the enum values in JsonPathItemType. I have instead committed a
different patch that adjusts the various switch cases to observe the
current order of the enum. That also means that the remaining patches
that add new item methods need to add the new enum values at the end and
adjust the rest of their code accordingly.