Tom Lane wrote:
> Bruce Momjian <[EMAIL PROTECTED]> writes:
> > Tom Lane wrote:
> >> Definitely *not*; I don't want to kluge up every call to SearchSysCache
> >> with a feature that's only relevant to a small number of them.
> 
> > Uh, then what?
> 
> Then we make a wrapper function.  Something like
> 
>       GetUndeletedColumnByName(relid,attname)
> 
> replaces SearchSysCache(ATTNAME,...) in those places where you don't
> want to see deleted columns.  It'd return NULL if it finds a column
> tuple but sees it's deleted.
> 
>       GetUndeletedColumnByNum(relid,attnum)
> 
> replaces SearchSysCache(ATTNUM,...) similarly.

Good idea.

> > My only other idea is to make a syscache that is like ATTNAME except
> > that it doesn't return a dropped column.
> 
> That would mean duplicate storage of tuples inside the catcache...

No, I was thinking of something that did the normal ATTNAME lookup in
the syscache code, then returned NULL on dropped columns;  similar to
your idea but done inside the syscache code rather than in a separate
function.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  [EMAIL PROTECTED]               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster

Reply via email to