Re: Fixes for missing schema qualifications
David G. Johnston <david.g.johnston@gmail.com>
From: "David G. Johnston" <david.g.johnston@gmail.com>
To: Tatsuo Ishii <ishii@sraoss.co.jp>
Cc: Noah Misch <noah@leadboat.com>, Michael Paquier <michael@paquier.xyz>, Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2018-03-14T00:19:50Z
Lists: pgsql-hackers
On Tue, Mar 13, 2018 at 5:11 PM, Tatsuo Ishii <ishii@sraoss.co.jp> wrote: > >>> + "select pg_catalog.count(*) " > >>> + "from pg_catalog.pg_namespace > where nspname = '%s'", > >> > >> This qualifies some functions, but it leaves plenty of unqualified > operators. > > Oops. I meant: > > select pg_catalog.count(*) from pg_catalog.pg_namespace where > pg_catalog.nameeq(nspname, '%s'); > > I'd rather write that: select [...] where nspname operator(pg_catalog.=) '%s' Introducing undocumented implementation functions to these queries is undesirable; and besides, indexing and equivalence relies on operators and not the underlying functions so there would be some risk of performance issues if the functions were used directly. David J.
Commits
-
Add some missing schema qualifications
- ee2b37ae044f 12.0 landed