GetTransactionSnapshot() in enum.c - Mailing list pgsql-hackers

From Andres Freund
Subject GetTransactionSnapshot() in enum.c
Date
Msg-id 20130819122938.GB8558@alap2.anarazel.de
Whole thread Raw
Responses Re: GetTransactionSnapshot() in enum.c  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Hi,

ISTM that we shouldn't use GetTransactionSnapshot() in enum.c but
GetLatestSnapshot() in <= 9.3 and NULL/GetCatalogSnapshot() > 9.3.

typecache.c's usage was converted to GetLatestSnapshot() but enum.c's
was not.

I don't seem to have full mental capacity right now, but I think the
current usage could cause problems with a range type index over a enum
column. Index predicates might also be problematic.

Greetings,

Andres Freund

-- Andres Freund                       http://www.2ndQuadrant.com/PostgreSQL Development, 24x7 Support, Training &
Services



pgsql-hackers by date:

Previous
From: Albe Laurenz
Date:
Subject: Re: LDAP: bugfix and deprecated OpenLDAP API
Next
From: Charles Sheridan
Date:
Subject: Automatic Index Creation for Column Types