Re: 2019-03 CF Summary / Review - Tranche #2 - Mailing list pgsql-hackers

From James Coleman
Subject Re: 2019-03 CF Summary / Review - Tranche #2
Date
Msg-id CAAaqYe-sO5EuA=HY6pWCSaJX_EC5ChsNb9pwNbbhbOLstj0U3Q@mail.gmail.com
Whole thread Raw
In response to Re: 2019-03 CF Summary / Review - Tranche #2  (Andres Freund <andres@anarazel.de>)
Responses Re: 2019-03 CF Summary / Review - Tranche #2  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> - Prove IS NOT NULL inference for large arrays
>
>   NR: No idea.

As the fairly new author of this patch, my perspective is that this
patch got quite a bit of review, albeit without a formal "yes or no"
response.

I'm obviously interested in getting it committed, and I believe it's a
fairly simple patch to look at (though since it's in predtest probably
requires extra brain cycles to be careful we don't make spurious
assumptions in the optimizer). It's also an obvious performance win
for queries that can use partial indexes with almost no additional
optimizer overhead.

But I'm also interested in feedback on how patches like this work in
the review process -- particularly when given the fair amount of
discussion/cleanup but without a final word. As a new patch author a
lot of understanding how this works feels very much like a "learn as
you go", but since there's not a lot of information directly written
on it I figured asking explicitly is the best way to learn the process
better.

Thanks,
James Coleman


pgsql-hackers by date:

Previous
From: Justin Pryzby
Date:
Subject: Re: REL_11_STABLE: dsm.c - cannot unpin a segment that is not pinned
Next
From: Michael Banck
Date:
Subject: Re: Progress reporting for pg_verify_checksums