RE: Bug in error reporting for multi-line JSON

From: "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: Hamid Akhtar <hamid(dot)akhtar(at)gmail(dot)com>, Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com>, "pgsql-bugs(at)postgresql(dot)org" <pgsql-bugs(at)postgresql(dot)org>
Subject: RE: Bug in error reporting for multi-line JSON
Date: 2021-08-25 08:22:57
Message-ID: OS0PR01MB61137C4054774F44E3A9DC89FBC69@OS0PR01MB6113.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Tuesday, March 2, 2021 6:50 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
>> I'm changing the status of this patch to Ready for Committer.
>
>I reviewed this and pushed it, with some changes.

I think the while condition "context_start < context_end" added in commit ffd3944ab9 is useless. Thoughts?

code added in ffd3944ab9
+ while (context_end - context_start >= 50 && context_start < context_end)

Regards,
Tang

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Daniel Gustafsson 2021-08-25 08:56:31 Re: Bug in error reporting for multi-line JSON
Previous Message Дилян Палаузов 2021-08-25 07:07:40 Re: No libldap_r in OpenLDAP 2.5 - use LDAP_API_FEATURE_X_OPENLDAP_REENTRANT from ldap_features.h

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-08-25 08:31:37 Re: row filtering for logical replication
Previous Message Andres Freund 2021-08-25 08:20:03 Re: pgstat_send_connstats() introduces unnecessary timestamp and UDP overhead