From: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Joe Conway <mail(at)joeconway(dot)com> |
Subject: | Re: [HACKERS] Macros bundling RELKIND_* conditions |
Date: | 2018-12-21 02:20:04 |
Message-ID: | CAExHW5s+MihMrGmtrUXzR7b6DV2uminqRcgXDw_FVjmHi3VL9Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Dec 19, 2018 at 11:37 PM Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
wrote:
> On 2017-Aug-02, Tom Lane wrote:
>
> > I think Peter's got the error and the detail backwards. It should be
> > more like
> >
> > ERROR: "someview" cannot have constraints
> > DETAIL: "someview" is a view.
> >
> > If we do it like that, we need one ERROR message per error reason,
> > and one DETAIL per relkind, which should be manageable.
>
> I support this idea. Here's a proof-of-concept patch that corresponds
> to one of the cases that Ashutosh was on about (specifically, the one
> that uses the RELKIND_CAN_HAVE_STORAGE macro I just added). If there
> are no objections to this approach, I'm going to complete it along these
> lines.
>
> I put the new function at the bottom of heapam.c but I think it probably
> needs a better place.
>
> BTW are there other opinions on the RELKIND_HAS_STORAGE vs.
> RELKIND_CAN_HAVE_STORAGE debate? I'm inclined to change it to the
> former.
>
+1 I liked the idea.
--
Best Wishes,
Ashutosh Bapat
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2018-12-21 03:19:21 | Re: Change pgarch_readyXlog() to return .history files first |
Previous Message | Yotsunaga, Naoki | 2018-12-21 01:47:40 | RE: [Proposal] Add accumulated statistics |