Re: Patch to support SEMI and ANTI join removal - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Patch to support SEMI and ANTI join removal
Date
Msg-id CAB7nPqSfbRndVQQCt+eVHZ1VcFqFoEJHdiiioXvxG1=okwRDcg@mail.gmail.com
Whole thread Raw
In response to Re: Patch to support SEMI and ANTI join removal  (Marko Tiikkaja <marko@joh.to>)
Responses Re: Patch to support SEMI and ANTI join removal  (Andres Freund <andres@2ndquadrant.com>)
List pgsql-hackers
<div dir="ltr"><br /><div class="gmail_extra"><br /><div class="gmail_quote">On Fri, Feb 13, 2015 at 4:57 PM, Marko
Tiikkaja<span dir="ltr"><<a href="mailto:marko@joh.to" target="_blank">marko@joh.to</a>></span> wrote:<br
/><blockquoteclass="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex"><divclass=""><div class="h5">On 2/13/15 8:52 AM, Michael Paquier wrote:<br
/><blockquoteclass="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">On Sun, Nov 23, 2014 at 8:23 PM, David Rowley <<a
href="mailto:dgrowleyml@gmail.com"target="_blank">dgrowleyml@gmail.com</a>> wrote:<br /><blockquote
class="gmail_quote"style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> As the
patchstands there's still a couple of FIXMEs in there, so there's<br /> still a bit of work to do yet.<br /> Comments
arewelcome<br /><br /></blockquote><br /> Hm, if there is still work to do, we may as well mark this patch as<br />
rejectedas-is, also because it stands in this state for a couple of months.<br /></blockquote><br /></div></div> I
didn'tbring this up before, but I'm pretty sure this patch should be marked "returned with feedback".  From what I've
understood,"rejected" means "we don't want this thing, not in this form or any other".  That doesn't seem to be the
casefor this patch, nor for a few others marked "rejected" in the currently in-progress commit fest.<span
class=""><fontcolor="#888888"><br /></font></span></blockquote></div><br /></div><div class="gmail_extra">In the new CF
app,marking a patch as "returned this feedback" adds it automatically to the next commit fest. And note that it is
actuallywhat I did for now to move on to the next CF in the doubt:<br /><a
href="https://commitfest.postgresql.org/3/27/">https://commitfest.postgresql.org/3/27/</a><br/></div><div
class="gmail_extra">Butif nothing is done, we should as well mark it as "rejected". Not based on the fact that it is
rejectedbased on its content, but to not bloat the CF app with entries that have no activity for months.<br clear="all"
/></div><divclass="gmail_extra">-- <br /><div class="gmail_signature">Michael<br /></div></div></div> 

pgsql-hackers by date:

Previous
From: Martijn van Oosterhout
Date:
Subject: Re: Manipulating complex types as non-contiguous structures in-memory
Next
From: Michael Paquier
Date:
Subject: Re: SSL information view