Re: commitfest app bug/feature - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: commitfest app bug/feature
Date
Msg-id CABUevEwV4MPs-rLyD+wm_VuBc9-MkPn2b6+3Tw8rKZQB9WBvtQ@mail.gmail.com
Whole thread Raw
In response to commitfest app bug/feature  (Fabien COELHO <coelho@cri.ensmp.fr>)
List pgsql-hackers
On Thu, May 7, 2015 at 9:17 PM, Fabien COELHO <coelho@cri.ensmp.fr> wrote:

Hello,

Some comments/bug report about the commitfest web app which just bite me:-)

When creating a patch with the commitfest app, if the thread does not exist the application seems to reject the creation, so you may resubmit because you know the thread is going to exist soon... Well, this is a bad idea:-)

Indeed the first submission, although it looked like it had failed, has really been created anyway, and the resubmission creates another entry. If you click again, you get a third. And so on.

Clearly I'm very slow at some things, but at least I've just pushed a fix for this bug now. It validates the thread exists *before* it creates the entry now...


--

pgsql-hackers by date:

Previous
From: Thomas Munro
Date:
Subject: Re: query in PostgreSQL
Next
From: Simon Riggs
Date:
Subject: Re: Test code is worth the space