[PATCHES] txid cleanup

2007-10-19 Thread Marko Kreen
Per Tom comments, attached patch does following: - Binary I/O support - Use ActiveSnapshot instead of SerializableSnapshot - Clarify function purposes - remove dev targets from Makefile There should not be any code depending on SerializableSnapshot, if there is thats a bug that needs fixing. Fi

Re: [PATCHES] txid cleanup

2007-10-11 Thread Marko Kreen
On 10/11/07, Tom Lane <[EMAIL PROTECTED]> wrote: > "Marko Kreen" <[EMAIL PROTECTED]> writes: > > Also - either Tom or Jan has changed the code to allow > > xmin == xmax. Is such case really valid? > > Yes, at least in CVS HEAD. Aha, I had missed that. Thanks for confirmation. -- marko ---

Re: [PATCHES] txid cleanup

2007-10-11 Thread Tom Lane
"Marko Kreen" <[EMAIL PROTECTED]> writes: > Per Tom comments, attached patch does following: > - Binary I/O support > - Use ActiveSnapshot instead of SerializableSnapshot > - Clarify function purposes > - remove dev targets from Makefile Applied with some further editorialization.

Re: [PATCHES] txid cleanup

2007-10-11 Thread Tom Lane
"Marko Kreen" <[EMAIL PROTECTED]> writes: > Also - either Tom or Jan has changed the code to allow > xmin == xmax. Is such case really valid? Yes, at least in CVS HEAD. regards, tom lane ---(end of broadcast)--- TIP 5: don'

[PATCHES] txid cleanup

2007-10-11 Thread Marko Kreen
Per Tom comments, attached patch does following: - Binary I/O support - Use ActiveSnapshot instead of SerializableSnapshot - Clarify function purposes - remove dev targets from Makefile There should not be any code depending on SerializableSnapshot, if there is thats a bug that needs fixing. Fi