Re: [BUGS] pg_tables view definition incorrect??

Started by Mike Quinnalmost 23 years ago2 messagesgeneral
Jump to latest
#1Mike Quinn
mquinn@co.merced.ca.us

The inclusion of the relkind='s' made me think that something other than tables
were to be included as well. I don't see any problem with the original intention
of returning only tables. The view now is operating "almost" as designed. It does
return "pg_xactlock" which is a special relation. The relkind='s' should be removed
in order to conform with the intention.

Mike Quinn

Tom Lane <tgl@sss.pgh.pa.us> 7/10/03 8:24:43 AM >>>

"Mike Quinn" <mquinn@co.merced.ca.us> writes:

Given that 'S' => Sequence and 's' => special shouldn't the last condition in the WHERE clause be:
c.relkind = 'S'::"char"

Only if you suppose that pg_tables should show sequences, which was not
the intention. (I'm not real sure why 's' relations are listed, since
they're not really tables either, but the view is operating as
designed.)

I'm prepared to listen to an argument that pg_tables should show
sequences, but that's a definition change not a bug fix. Want to
bring it up on -general or -hackers?

regards, tom lane

#2Bruce Momjian
bruce@momjian.us
In reply to: Mike Quinn (#1)
Re: [GENERAL] [BUGS] pg_tables view definition incorrect??

OK, sequences removed from pg_tables. Regression updated. Patch
applied.

---------------------------------------------------------------------------

Mike Quinn wrote:

The inclusion of the relkind='s' made me think that something other than tables
were to be included as well. I don't see any problem with the original intention
of returning only tables. The view now is operating "almost" as designed. It does
return "pg_xactlock" which is a special relation. The relkind='s' should be removed
in order to conform with the intention.

Mike Quinn

Tom Lane <tgl@sss.pgh.pa.us> 7/10/03 8:24:43 AM >>>

"Mike Quinn" <mquinn@co.merced.ca.us> writes:

Given that 'S' => Sequence and 's' => special shouldn't the last condition in the WHERE clause be:
c.relkind = 'S'::"char"

Only if you suppose that pg_tables should show sequences, which was not
the intention. (I'm not real sure why 's' relations are listed, since
they're not really tables either, but the view is operating as
designed.)

I'm prepared to listen to an argument that pg_tables should show
sequences, but that's a definition change not a bug fix. Want to
bring it up on -general or -hackers?

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
(send "unregister YourEmailAddressHere" to majordomo@postgresql.org)

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

Attachments:

/bjm/difftext/plainDownload+4-4