Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Michael Paquier
On Mon, Aug 31, 2015 at 10:31 PM, Andres Freund wrote: > On 2015-08-31 16:22:54 +0300, Alexander Korotkov wrote: >> Is it correct to switch 2015-09 commitfest to inprogress now? No, this was not correct, it is expected to begin tomorrow. > Yea, isn't it only starting the 15th? If my memory does

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Andreas Karlsson
On 08/31/2015 03:34 PM, Alexander Korotkov wrote: +1 for making these dates more explicit Yeah, I think being explicit would make life easier for newcomers. Andreas -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgres

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Magnus Hagander
On Mon, Aug 31, 2015 at 3:34 PM, Alexander Korotkov < a.korot...@postgrespro.ru> wrote: > On Mon, Aug 31, 2015 at 4:31 PM, Andres Freund wrote: > >> On 2015-08-31 16:22:54 +0300, Alexander Korotkov wrote: >> > Is it correct to switch 2015-09 commitfest to inprogress now? >> >> Yea, isn't it only

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Alexander Korotkov
On Mon, Aug 31, 2015 at 4:31 PM, Andres Freund wrote: > On 2015-08-31 16:22:54 +0300, Alexander Korotkov wrote: > > Is it correct to switch 2015-09 commitfest to inprogress now? > > Yea, isn't it only starting the 15th? AFICS, on the last developer meeting it was decided to start commitfests in

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Andres Freund
On 2015-08-31 16:22:54 +0300, Alexander Korotkov wrote: > Is it correct to switch 2015-09 commitfest to inprogress now? Yea, isn't it only starting the 15th? Can we add an option to display days in the CF app? Greetings, Andres Freund -- Sent via pgsql-hackers mailing list (pgsql-hackers@post

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Alexander Korotkov
On Mon, Aug 31, 2015 at 3:16 PM, Michael Paquier wrote: > On Mon, Aug 31, 2015 at 9:13 PM, Magnus Hagander wrote: > > Anyway - that CF is closed, but we seem to not have *any* CF open at this > > point. Should we not make 2015-11 open? > > Yeah, switched. Is it correct to switch 2015-09 commitf

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Michael Paquier
On Mon, Aug 31, 2015 at 9:13 PM, Magnus Hagander wrote: > Anyway - that CF is closed, but we seem to not have *any* CF open at this > point. Should we not make 2015-11 open? Yeah, switched. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-31 Thread Magnus Hagander
On Sun, Aug 30, 2015 at 2:53 PM, Michael Paquier wrote: > > > And the commit fest of 2015-07 is now closed with the following score: > Committed: 58. > Moved to next CF: 25. > Rejected: 9. > Returned with Feedback: 25. > Total: 117. > Thanks! > > Ugh. Good to have it closed, but it seems we're st

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-30 Thread Michael Paquier
On Wed, Aug 26, 2015 at 4:59 PM, Michael Paquier wrote: > On Tue, Aug 25, 2015 at 4:39 PM, Michael Paquier > wrote: > > On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas > wrote: > >> Hello Hackers, > >> > >> There are a few "Needs Review" items remaining in the July commitfest. > >> Reviewer

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-26 Thread Michael Paquier
On Tue, Aug 25, 2015 at 4:39 PM, Michael Paquier wrote: > On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas wrote: >> Hello Hackers, >> >> There are a few "Needs Review" items remaining in the July commitfest. >> Reviewers, please take action - you are holding up the commitfest. >> >> In additi

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Michael Paquier
On Wed, Aug 26, 2015 at 10:30 AM, Andreas Karlsson wrote: > On 08/25/2015 09:39 AM, Michael Paquier wrote: >> >> -- Reload SSL certificates on SIGHUP: returned with feedback? I think >> that this patch needs more work to be in a commitable state. > > > Maybe I am being dense here, but I do not fee

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Andreas Karlsson
On 08/25/2015 09:39 AM, Michael Paquier wrote: -- Reload SSL certificates on SIGHUP: returned with feedback? I think that this patch needs more work to be in a commitable state. Maybe I am being dense here, but I do not feel like I have gotten any clear feedback which gives me a way forward wi

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Michael Paquier
On Tue, Aug 25, 2015 at 11:28 PM, Stephen Frost wrote: > Michael, > > * Michael Paquier (michael.paqu...@gmail.com) wrote: >> -- Default Roles: Stephen, are you planning to work on that for next CF? > > Yup! OK. Fine for me. I have moved the patch to next CF, even if I mentioned having marked it

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Heikki Linnakangas
On 08/25/2015 10:39 AM, Michael Paquier wrote: On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas wrote: Hello Hackers, There are a few "Needs Review" items remaining in the July commitfest. Reviewers, please take action - you are holding up the commitfest. In addition to these items, there

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Stephen Frost
Michael, * Michael Paquier (michael.paqu...@gmail.com) wrote: > -- Default Roles: Stephen, are you planning to work on that for next CF? Yup! Thanks! Stephen signature.asc Description: Digital signature

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Fabrízio de Royes Mello
On Tue, Aug 25, 2015 at 4:39 AM, Michael Paquier wrote: > > [...] > > -- Support to COMMENT ON CURRENT DATABASE: returned with feedback? The > author mentioned that patch will be reworked but there has been no new > version around it seems. > Moved to the next commitfest. Regards, -- Fabrízio d

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Joe Conway
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/25/2015 12:39 AM, Michael Paquier wrote: > -- self-defined policy for sepgsql regression test, returned with > feedback? The regressions are broken as mentioned at the end of > the thread. I am in the process of getting a VM setup with an appro

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Tomas Vondra
Hi, On 08/25/2015 02:44 PM, Michael Paquier wrote: On Tue, Aug 25, 2015 at 6:05 PM, Fabien COELHO wrote: -- merging pgbench logs: returned with feedback or bump? Fabien has concerns about performance regarding fprintf when merging the logs. Fabien, Tomas, thoughts? -- pgbench - per-transacti

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Michael Paquier
On Tue, Aug 25, 2015 at 6:05 PM, Fabien COELHO wrote: > >> -- merging pgbench logs: returned with feedback or bump? Fabien has >> concerns about performance regarding fprintf when merging the logs. >> Fabien, Tomas, thoughts? >> -- pgbench - per-transaction and aggregated logs: returned with >> fe

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Michael Paquier
On Tue, Aug 25, 2015 at 5:51 PM, Heikki Linnakangas wrote: > On 08/25/2015 10:39 AM, Michael Paquier wrote: >> - 12 patches are waiting on author: > > These can all be marked as Returned with good conscience, they've gotten at > least some feedback. Fine for me. I'll notify each thread individuall

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Fabien COELHO
-- merging pgbench logs: returned with feedback or bump? Fabien has concerns about performance regarding fprintf when merging the logs. Fabien, Tomas, thoughts? -- pgbench - per-transaction and aggregated logs: returned with feedback or bump to next CF? Fabien, Tomas, thoughts? I think that bo

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-25 Thread Michael Paquier
On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas wrote: > Hello Hackers, > > There are a few "Needs Review" items remaining in the July commitfest. > Reviewers, please take action - you are holding up the commitfest. > > In addition to these items, there are a bunch of items in "Ready for > Com

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-11 Thread Stephen Frost
Heikki, * Heikki Linnakangas (hlinn...@iki.fi) wrote: > >* self-defined policy for sepgsql regression test > > Stephen: would you have the time to handle this, please? We'll definitely get this addressed soon. Apologies for taking so long to get to it, was in Brazil last week and over the weeke

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Fabien COELHO
* extends pgbench expressions with functions Robert signed up as reviewer for this a long time ago. Ping, do you still plan to review this? I seem to recall that I nominated Robert when adding the patch, because this is an extension of his expression patch. So the "sign up" is really just a

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Etsuro Fujita
On 2015/08/10 17:10, Ashutosh Bapat wrote: On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas mailto:hlinn...@iki.fi>> wrote: * Join pushdown support for foreign tables Ashutosh, KaiGai, Etsuro: You signed up as reviewers in spring, but there hasn't been any activity during th

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Ashutosh Bapat
On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas wrote: > Hello Hackers, > > There are a few "Needs Review" items remaining in the July commitfest. > Reviewers, please take action - you are holding up the commitfest. > > In addition to these items, there are a bunch of items in "Ready for > Co

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Heikki Linnakangas
On 08/10/2015 10:46 AM, Atri Sharma wrote: * Unique Joins Still needs to be reviewed. Any volunteers? Can take this one up, if its within my limits. Thanks! I've added you as reviewer to that. - Heikki -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Amit Kapila
On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas wrote: > > * Parallel Seq scan >> > > Jeff, Haribabu, you're signed up as reviewers. How's it going? > The current state of the patch is that there are few comments by Jeff and others which I need to take care, however it would have been better

Re: [HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Atri Sharma
> > * Unique Joins >> > > Still needs to be reviewed. Any volunteers? > > > Can take this one up, if its within my limits.

[HACKERS] Commitfest remaining "Needs Review" items

2015-08-10 Thread Heikki Linnakangas
Hello Hackers, There are a few "Needs Review" items remaining in the July commitfest. Reviewers, please take action - you are holding up the commitfest. In addition to these items, there are a bunch of items in "Ready for Committer" state. Committers: please help with those. * extends pgbe