On Wed, Mar 28, 2018 at 7:17 PM, Andres Freund <andres@anarazel.de> wrote:
On 2018-03-28 16:28:01 +0300, Teodor Sigaev wrote: > > BTW, patch had conflicts with master. Please, find rebased version attached. > > Despite by patch conflist patch looks commitable, has anybody objections to > commit it?
> Patch recieved several rounds of review during 2 years, and seems to me, > keeping it out from sources may cause a lost it. Although it suggests > performance improvement in rather wide usecases.
My impression it has *NOT* received enough review to be RFC. Not saying it's impossible to get there this release, but that just committing it doesn't seem wise.
I would say that executor part of this patch already received plenty of review.
For sure, there still might be issues. I just mean that amount of review of
executor part of this patch is not less than in average patch we commit.
But optimizer and costing part of this patch still need somebody to take