From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Amit Langote <amitlangote09(at)gmail(dot)com> |
Cc: | Chris Bandy <bandy(dot)chris(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: [PATCH] Add schema and table names to partition error |
Date: | 2020-03-20 06:52:01 |
Message-ID: | CAA4eK1+7OBfrnA8tV0AWGB+gpNqDjA4+qMP7RZPj40tA8ppGQQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Mar 19, 2020 at 3:34 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
>
> Thank you Chris, Amit.
>
> On Thu, Mar 19, 2020 at 1:46 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > On Thu, Mar 19, 2020 at 3:55 AM Chris Bandy <bandy(dot)chris(at)gmail(dot)com> wrote:
> > >
> > >
> > > Sorry for these troubles. Attached are patches created using `git
> > > format-patch -n -v6` on master at 487e9861d0.
> > >
> >
> > No problem. I have extracted your code changes as a separate patch
> > (see attached) as I am not sure we want to add tests for these cases.
> > This doesn't apply in back-branches, but I think that is small work
> > and we can do that if required. The real question is do we want to
> > back-patch this? Basically, this improves the errors in certain cases
> > by providing additional information that otherwise the user might need
> > to extract from error messages. So, there doesn't seem to be pressing
> > need to back-patch this but OTOH, we have mentioned in docs that we
> > support to display this information for all SQLSTATE class 23
> > (integrity constraint violation) errors which is not true as we forgot
> > to adhere to that in some parts of code.
> >
> > What do you think? Anybody else has an opinion on whether to
> > back-patch this or not?
>
> As nobody except Chris complained about this so far, maybe no?
>
Fair enough, unless I see any other opinions, I will push this on Monday.
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2020-03-20 06:59:01 | Re: Berserk Autovacuum (let's save next Mandrill) |
Previous Message | Justin Pryzby | 2020-03-20 06:51:20 | Re: error context for vacuum to include block number |