Re: Patch to support SEMI and ANTI join removal

<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>