Re: Moving snapshot code around - Mailing list pgsql-patches

From Alvaro Herrera
Subject Re: Moving snapshot code around
Date
Msg-id 20080326211412.GQ29730@alvh.no-ip.org
Whole thread Raw
In response to Re: Moving snapshot code around  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-patches
Tom Lane wrote:
> Alvaro Herrera <alvherre@commandprompt.com> writes:
> > Tom Lane wrote:

> >> I don't like that very much.  What about just moving the HTSU_Result
> >> enum's declaration somewhere else?  Two possibilities are heapam.h
> >> itself, or the new snapshot.h file (which'd then have to be included
> >> by heapam.h, but it seems lightweight enough that that's not too
> >> terrible).
>
> > Well, heapam.h includes a lot of other headers, so it doesn't look a
> > good candidate to me.  I think snapshot.h is a reasonably good
> > candidate.
>
> Works for me.

This part done too.

Thanks for the input.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: V1.1 patch for TODO Item: SQL-language reference parameters by name.
Next
From: Tom Lane
Date:
Subject: Re: Fix pg_dump dependency on postgres.h