Re: pgsql/src/backend/storage/smgr (smgr.c) - Mailing list pgsql-committers

From Hiroshi Inoue
Subject Re: pgsql/src/backend/storage/smgr (smgr.c)
Date
Msg-id 3A107F37.A883C5E4@tpf.co.jp
Whole thread Raw
In response to pgsql/src/backend/storage/smgr (smgr.c)  (inoue@postgresql.org)
List pgsql-committers
Tom Lane wrote:

> inoue@postgresql.org writes:
> > prevent open failure of VIEW
>
> I do not like this fix.  We should be moving away from having smgr
> access relcache entries, not adding more relcache knowledge to it.
> So checks for view-ness should be at a higher level.
>
> What case were you seeing a failure in?
>

I see the failure in xlogutils.c.
Oops I'm mistaken,another is smgrnblocks call
in RelationGetNumberOfBlocks() and I didn't
change it at all.
I don't like this fix either and I intend to change
at higher level first. However it seems to me
that checking relkind every time is not only ugly
but also unreliable.

I don't stick to my change.
Please change as you prefer.

Regards.
Hiroshi Inoue



pgsql-committers by date:

Previous
From: momjian@postgresql.org
Date:
Subject: pgsql/src/interfaces/libpq (fe-connect.c libpq-int.h)
Next
From: momjian@postgresql.org
Date:
Subject: pgsql/doc/src/sgml (libpq.sgml)