From: | "Cui Shijun" <rancpine(at)gmail(dot)com> |
---|---|
To: | "Bruce Momjian" <bruce(at)momjian(dot)us> |
Cc: | "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>, "Dave Page" <dpage(at)postgresql(dot)org>, "Marc G(dot) Fournier" <scrappy(at)hub(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Lack of urgency in 8.3 reviewing |
Date: | 2007-05-17 14:36:23 |
Message-ID: | 306760850705170736r11b70550ged4d6db302b54671@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I see...
I checked part of HOT patches(patch1), and found that it involves too
many things I am not currently familar with. Maybe I should change an
item to work. :-(
Since I only studied part of source codes about transaction
processing(lmgr/MVCC/xact but without xlog.c), I want to study
"Group Commit" patch and try to review it, any suggestions?
2007/5/17, Bruce Momjian <bruce(at)momjian(dot)us>:
> Pavan Deolasee wrote:
> > On 5/17/07, Cui Shijun <rancpine(at)gmail(dot)com> wrote:
> > >
> > > I want to help the reviewing work of "ctid chain following enhancement ".
> > > I've been studying the souce code which related with that part recently.
> > > :-)
> >
> >
> >
> > Tom had objected to this patch on the grounds that it adds complexity
> > without any significant gains. Though I don't completely agree with the
> > first part, the second part is indeed debatable since the code is touched
> > only
> > for infrequently.
>
> Right. The reason the patch was kept in the queue is that there was
> discussion that HOT will exercise that part of the code a lot more than
> it does currently.
>
> --
> Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
> EnterpriseDB http://www.enterprisedb.com
>
> + If your life is a hard drive, Christ can be your backup. +
>
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2007-05-17 14:39:56 | Re: Not ready for 8.3 |
Previous Message | Tom Lane | 2007-05-17 14:31:59 | Re: CREATE TABLE LIKE INCLUDING INDEXES support |