Re: Report error position in partition bound check

From: Alexandra Wang <walexandra(at)vmware(dot)com>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>, "Alexandra Wang (Pivotal)" <lewang(at)pivotal(dot)io>
Cc: Ashutosh Bapat <ashutosh(dot)bapat(at)2ndquadrant(dot)com>, Amit Langote <amitlangote09(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>, "Ashwin Agrawal (Pivotal)" <aagrawal(at)pivotal(dot)io>
Subject: Re: Report error position in partition bound check
Date: 2020-07-13 17:53:46
Message-ID: DM6PR05MB692465B221C78E971542F745B0600@DM6PR05MB6924.namprd05.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 2 July 2020, at 06:39, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> > On 10 Apr 2020, at 23:50, Alexandra Wang <lewang(at)pivotal(dot)io> wrote:
>
> > On Fri, Apr 10, 2020 at 8:37 AM Ashutosh Bapat <ashutosh(dot)bapat(at)2ndquadrant(dot)com <mailto:ashutosh(dot)bapat(at)2ndquadrant(dot)com>> wrote:
> > > for a multi-key value the ^
> > > points to the first column and the reader may think that that's the
> > > problematci column. Should it instead point to ( ?
> >
> > I attached a v2 of Amit's 0002 patch to also report the exact column
> > for the partition overlap errors.
>
> This patch fails to apply to HEAD due to conflicts in the create_table expected
> output. Can you please submit a rebased version? I'm marking the CF entry
> Waiting on Author in the meantime.

Thank you Daniel. Here's the rebased patch. I also squashed the two
patches into one so it's easier to review.

--
Alex

Attachment Content-Type Size
v3-0001-Improve-check-new-partition-bound-error-position-rep.patch text/x-patch 23.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Pryzby 2020-07-13 17:59:57 Re: proposal: possibility to read dumped table's name from file
Previous Message Alvaro Herrera 2020-07-13 17:52:12 Re: min_safe_lsn column in pg_replication_slots view