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

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

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

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

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

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:

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

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

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

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

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

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

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

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

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

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

Re: [HACKERS] Commitfest remaining Needs Review items

On Wed, Aug 26, 2015 at 4:59 PM, Michael Paquier michael.paqu...@gmail.com wrote: On Tue, Aug 25, 2015 at 4:39 PM, Michael Paquier michael.paqu...@gmail.com wrote: On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas hlinn...@iki.fi wrote: Hello Hackers, There are a few Needs Review

Re: [HACKERS] Commitfest remaining Needs Review items

On Tue, Aug 25, 2015 at 4:39 PM, Michael Paquier michael.paqu...@gmail.com wrote: On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas hlinn...@iki.fi wrote: Hello Hackers, There are a few Needs Review items remaining in the July commitfest. Reviewers, please take action - you are holding up

Re: [HACKERS] Commitfest remaining Needs Review items

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

-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

Re: [HACKERS] Commitfest remaining Needs Review items

On Tue, Aug 25, 2015 at 4:39 AM, Michael Paquier michael.paqu...@gmail.com 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.

Re: [HACKERS] Commitfest remaining Needs Review items

On 08/25/2015 10:39 AM, Michael Paquier wrote: On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas hlinn...@iki.fi 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

Re: [HACKERS] Commitfest remaining Needs Review items

On Wed, Aug 26, 2015 at 10:30 AM, Andreas Karlsson andr...@proxel.se 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

Re: [HACKERS] Commitfest remaining Needs Review items

On Mon, Aug 10, 2015 at 4:34 PM, Heikki Linnakangas hlinn...@iki.fi 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

Re: [HACKERS] Commitfest remaining Needs Review items

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

Re: [HACKERS] Commitfest remaining Needs Review items

On Tue, Aug 25, 2015 at 11:28 PM, Stephen Frost sfr...@snowman.net 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

Re: [HACKERS] Commitfest remaining Needs Review items

On Tue, Aug 25, 2015 at 6:05 PM, Fabien COELHO coe...@cri.ensmp.fr 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

Re: [HACKERS] Commitfest remaining Needs Review items

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

Re: [HACKERS] Commitfest remaining Needs Review items

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

Re: [HACKERS] Commitfest remaining Needs Review items

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

Re: [HACKERS] Commitfest remaining Needs Review items

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 weekend

Re: [HACKERS] Commitfest remaining Needs Review items

* 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

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

Re: [HACKERS] Commitfest remaining Needs Review items

On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas hlinn...@iki.fi 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

[HACKERS] Commitfest remaining Needs Review items

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 pgbench

Re: [HACKERS] Commitfest remaining Needs Review items

* Unique Joins Still needs to be reviewed. Any volunteers? Can take this one up, if its within my limits.

Re: [HACKERS] Commitfest remaining Needs Review items

On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas hlinn...@iki.fi 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

Re: [HACKERS] Commitfest remaining Needs Review items

On 2015/08/10 17:10, Ashutosh Bapat wrote: On Mon, Aug 10, 2015 at 1:04 PM, Heikki Linnakangas hlinn...@iki.fi 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