Horton

Re: duplicate line in ExecEvalJsonCoercionFinish

Richard Guo <guofenglinux@gmail.com>

View thread View on postgr.es
From: Richard Guo <guofenglinux@gmail.com>
To: Tender Wang <tndrwang@gmail.com>
Cc: jian he <jian.universality@gmail.com>, Amit Langote <amitlangote09@gmail.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2025-07-17T01:01:26Z
Lists: pgsql-hackers 
On Thu, Jul 17, 2025 at 9:52 AM Tender Wang <tndrwang@gmail.com> wrote:
> jian he <jian.universality@gmail.com> 于2025年7月17日周四 07:39写道：
>> ExecEvalJsonCoercionFinish duplicate line:
>>         jsestate->escontext.error_occurred = false;
>>         jsestate->escontext.error_occurred = false;

> Good catch.  It was introduced by this commit 231b7d670b21.
> Add Amit to the cc list.

Nice catch!  The two duplicate statements were introduced in separate
commits; looks like a copy-paste oversight in the later one.

Thanks
Richard

Commits

  1. Remove duplicate line

  2. SQL/JSON: Improve error-handling of JsonBehavior expressions