Re: HOT patch - version 13

From: Decibel! <decibel(at)decibel(dot)org>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>
Cc: Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: HOT patch - version 13
Date: 2007-08-07 18:00:37
Message-ID: 20070807180036.GI25704@nasby.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 07, 2007 at 06:53:48PM +0100, Simon Riggs wrote:
> On Tue, 2007-08-07 at 18:25 +0530, Pavan Deolasee wrote:
>
> > I am posting the version 13 of HOT patch on pgsql-patches. It
> > includes the changes based on feedback received on version 11.
>
> I think we need a change log, so we can see which feedback has resulted
> in changes and which has not. Otherwise we'll end up needing to re-read
> the patch frequently. I'll be straight up and say I don't read every
> version you put out, especially in recent months. You can really expect
> most people to eyeball it only once, so we need a way to ensure every
> reviewer's comment that results in a change is tracked.
>
> I wish I had done this myself with async commit, though the
> changes/suggestions were relatively modest in comparison to HOT.
>
> Perhaps if we start that from now on? That will help as we move towards
> final reviews and commit. We seem to be discussing smaller aspects of
> the patch now.

Perhaps also posting a diff of the patch versions would help, so that
you can easily see exactly what code has changed since last time...
--
Decibel!, aka Jim Nasby decibel(at)decibel(dot)org
EnterpriseDB http://enterprisedb.com 512.569.9461 (cell)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavan Deolasee 2007-08-07 18:01:44 Re: HOT patch - version 11
Previous Message Simon Riggs 2007-08-07 17:53:48 Re: HOT patch - version 13