From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-hackers(at)postgresql(dot)org, andrew(at)supernews(dot)com |
Subject: | Re: slow information schema with thausand users, seq.scan pg_authid |
Date: | 2006-02-06 18:29:10 |
Message-ID: | 28046.1139250550@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
> Andrew - Supernews wrote:
>> Perhaps you missed the fact that the query was not one that he wrote,
>> but is the query that psql uses for \ds ?
> I did miss that. Perhaps with dependency tracking and all, we don't
> need the left joins anymore?
I don't see anything wrong with leaving the left joins as-is, on the
grounds that
1. the planner can simplify the left joins to inner joins, eg the
join to pg_namespace should be simplified on the strength of the
test on nspname. (This seems to be broken in HEAD, but it does
work in 8.1 --- I think I broke it with the changes to treat IN
as a ScalarArrayOp. Will fix.)
2. HEAD also knows how to change the order of the left joins at need.
The real question to me is why the planner doesn't want to use the
index on pg_authid.oid. That's pretty curious ...
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Scott Marlowe | 2006-02-06 18:37:53 | Re: autovacuum |
Previous Message | James William Pye | 2006-02-06 18:25:05 | Re: Copy From & Insert UNLESS |
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2006-02-06 18:55:20 | Re: [HACKERS] Krb5 & multiple DB connections |
Previous Message | Peter Eisentraut | 2006-02-06 17:21:46 | Re: slow information schema with thausand users, seq.scan pg_authid |