Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-26 Thread Amit Langote
On Wed, Nov 26, 2025 at 8:57 PM Kirill Reshke wrote: > On Wed, 26 Nov 2025 at 10:32, Amit Langote wrote: > > On Wed, Nov 19, 2025 at 11:55 AM jian he > > wrote: > > > On Fri, Nov 7, 2025 at 2:26 PM Kirill Reshke > > > wrote: > > > > > > > > Hi! > > > > I tried your fix and this indeed fixes a

Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-26 Thread Kirill Reshke
On Wed, 26 Nov 2025 at 10:32, Amit Langote wrote: > > Hi, > > On Wed, Nov 19, 2025 at 11:55 AM jian he wrote: > > On Fri, Nov 7, 2025 at 2:26 PM Kirill Reshke wrote: > > > > > > Hi! > > > I tried your fix and this indeed fixes an issue. Two minor comments: > > > > > > First, > > > in the `src/ba

Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-26 Thread jian he
On Wed, Nov 26, 2025 at 3:32 PM Amit Langote wrote: > > > > First, > > > in the `src/backend/parser/parse_expr.c` fil there are multiple > > > examples of working with `coerce_to_target_type`, they all share > > > different coding practice: > > > > > > ``` > > > coerced_expr = coerce_to_target_typ

Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-25 Thread Amit Langote
Hi, On Wed, Nov 19, 2025 at 11:55 AM jian he wrote: > On Fri, Nov 7, 2025 at 2:26 PM Kirill Reshke wrote: > > > > Hi! > > I tried your fix and this indeed fixes an issue. Two minor comments: > > > > First, > > in the `src/backend/parser/parse_expr.c` fil there are multiple > > examples of workin

Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-18 Thread jian he
On Fri, Nov 7, 2025 at 2:26 PM Kirill Reshke wrote: > > Hi! > I tried your fix and this indeed fixes an issue. Two minor comments: > > First, > in the `src/backend/parser/parse_expr.c` fil there are multiple > examples of working with `coerce_to_target_type`, they all share > different coding prac

Re: transformJsonFuncExpr pathspec cache lookup failed

2025-11-06 Thread Kirill Reshke
On Fri, 7 Nov 2025 at 07:50, jian he wrote: > > hi. > > in transformJsonFuncExpr: > > path_spec = transformExprRecurse(pstate, func->pathspec); > path_spec = coerce_to_target_type(pstate, path_spec, exprType(path_spec), > JSONPATHOID, -1, >

transformJsonFuncExpr pathspec cache lookup failed

2025-11-06 Thread jian he
hi. in transformJsonFuncExpr: path_spec = transformExprRecurse(pstate, func->pathspec); path_spec = coerce_to_target_type(pstate, path_spec, exprType(path_spec), JSONPATHOID, -1, COERCION_EXPLICIT, COERCE_IMPLICIT