Patch Status

Started by Simon Riggsover 18 years ago3 messagespatches
Jump to latest
#1Simon Riggs
simon@2ndQuadrant.com

I've submitted 5 patches in Jan/Feb, none of which appear on the patch
status page for the CommitFest, or any other status page I'm aware of.
There *is* a comment on that page about "Minor Recovery..." which was a
patch from me that was committed to 8.3 on 26 Sep 2007, so that item can
be removed from the list.

My understanding of status on them is this:

* Bulk Insert tuning - 26 Feb
Reviewed and changes agreed. With me to complete patch.

* TransactionIdIsInProgress cache
Committed on 11 Mar

* Truncate Triggers - 30 Jan
Not reviewed yet.
This is a fully tested and production ready patch, apart from a few
sections of #ifdef'd code that were intended as stubs to show how we
might implement a brief suggestion of Tom's as an extension to the
concept of Truncate Triggers. I think we can ignore that and look to
apply the patch, minus the #ifdef'd sections now. Happy to produce a
clean patch with no #ifdef'd sections if that assists review.

* sinval contention reduction - 25 Jan
A discussion patch prior to some performance testing which subsequently
never took place. FWIW I have an updated patch which I haven't yet
published but haven't yet had access to a test machine to try this at
high volumes.

* Doc patch for bug 3877 - 17 Jan
Ready to be applied.

My understanding is that all prior patches of mine have been dealt with.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com

PostgreSQL UK 2008 Conference: http://www.postgresql.org.uk

#2Bruce Momjian
bruce@momjian.us
In reply to: Simon Riggs (#1)
Re: Patch Status

Well, this all seemed like work-in-progress stuff so it wasn't being
tracked. If you have things that need to be reviewed/applied, I suggest
you resubmit and mark them as ready.

As far as the doc patch, I see that as applied by Tom:

revision 2.113
date: 2008/01/23 20:21:37; author: tgl; state: Exp; lines: +10 -4
branches: 2.113.2;
Provide a concrete example of parameter expansion in archive_command.
Per discussion of bug #3877. Simon Riggs, some fixes by moi.

---------------------------------------------------------------------------

Simon Riggs wrote:

I've submitted 5 patches in Jan/Feb, none of which appear on the patch
status page for the CommitFest, or any other status page I'm aware of.
There *is* a comment on that page about "Minor Recovery..." which was a
patch from me that was committed to 8.3 on 26 Sep 2007, so that item can
be removed from the list.

My understanding of status on them is this:

* Bulk Insert tuning - 26 Feb
Reviewed and changes agreed. With me to complete patch.

* TransactionIdIsInProgress cache
Committed on 11 Mar

* Truncate Triggers - 30 Jan
Not reviewed yet.
This is a fully tested and production ready patch, apart from a few
sections of #ifdef'd code that were intended as stubs to show how we
might implement a brief suggestion of Tom's as an extension to the
concept of Truncate Triggers. I think we can ignore that and look to
apply the patch, minus the #ifdef'd sections now. Happy to produce a
clean patch with no #ifdef'd sections if that assists review.

* sinval contention reduction - 25 Jan
A discussion patch prior to some performance testing which subsequently
never took place. FWIW I have an updated patch which I haven't yet
published but haven't yet had access to a test machine to try this at
high volumes.

* Doc patch for bug 3877 - 17 Jan
Ready to be applied.

My understanding is that all prior patches of mine have been dealt with.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com

PostgreSQL UK 2008 Conference: http://www.postgresql.org.uk

--
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

#3Simon Riggs
simon@2ndQuadrant.com
In reply to: Bruce Momjian (#2)
Re: Patch Status

On Mon, 2008-03-17 at 12:54 -0400, Bruce Momjian wrote:

Well, this all seemed like work-in-progress stuff so it wasn't being
tracked. If you have things that need to be reviewed/applied, I suggest
you resubmit and mark them as ready.

OK thanks, will do.

As far as the doc patch, I see that as applied by Tom:

revision 2.113
date: 2008/01/23 20:21:37; author: tgl; state: Exp; lines: +10 -4
branches: 2.113.2;
Provide a concrete example of parameter expansion in archive_command.
Per discussion of bug #3877. Simon Riggs, some fixes by moi.

Yep, see it now. My mistake.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com

PostgreSQL UK 2008 Conference: http://www.postgresql.org.uk