Robert Haas wrote:
> On Thu, Oct 15, 2015 at 12:22 PM, Alvaro Herrera
> <alvherre@2ndquadrant.com> wrote:
> > Robert Haas wrote:
> >> - Oversize item computation needs more testing (c.f. ereport(ERROR)
> >> calls in brin_getinsertbuffer). This is pretty vague, and there's no
> >> thread linked. If there's a stability issue here, presumably it falls
> >> to Alvaro to fix it. But I don't know who added this item or what
> >> really needs to be done.
> >
> > I added it, sorry it's vague. It means that I should test with
> > values of increasing size and see if all the errors are correctly
> > covered, i.e. we don't get a PANIC because of a failure in PageAddItem.
>
> So, are you going to do that?
Yes.
> > I've been looking at this on and off, without anything useful to share
> > yet. One approach that was suggested (which I don't like much, but I
> > admit is a possible approach) is that we just need to remain vigilant
> > that all features that add DDL properly test the event trigger side of
> > it, just as we remain vigilant about pg_dump support without having any
> > explicit test that it works.
>
> I really, really hope that's not where we end up. Just shoot me now.
Me too. I'm going to do something about this, though I can't promise a
deadline.
> > The other item on me is the documentation patch by Emre Hasegeli for
> > usage of the inclusion opclass framework in BRIN. I think it needs some
> > slight revision by some native English speaker and I'm not completely in
> > love with the proposed third column in the table it adds, but otherwise
> > is factually correct as far as I can tell.
>
> I'm not clear whether you are asking for help with this, or ...?
I enlisted the help of Ian Barwick for this one.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services