On Sat, Apr 4, 2015 at 4:19 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote: > Hi > > 2015-03-31 14:38 GMT+02:00 Haribabu Kommi <kommi.haribabu@gmail.com>: >> >> keyword_databases - The database name can be "all", "replication", >> sameuser", "samerole" and "samegroup". >> keyword_roles - The role can be "all" and a group name prefixed with "+". > > > I am not very happy about name - but I have no better idea :( - maybe > "database_mask", "user_mask" >
How about database_keywords and user_keywords as column names?
I am thinking, it is better than keyword_databases - it is more logical. But it is maybe question for mathematicians.
some other variant "database_filters" and "user_filters" or "database_predicates" and "user_predicates"
but it is not terrible nice too
Regards
Pavel
>> >> >> The rest of the database and role names are treated as normal database >> and role names. >> >> 2. Added the code changes to identify the names with quoted. > > > Is it a good idea? Database's names are not quoted every else (in system > catalog). So now, we cannot to use join to this view. > > postgres=# select (databases)[1] from pg_hba_conf ; > databases > ----------- > "omega 2" > > (4 rows) > > postgres=# select datname from pg_database ; > datname > ----------- > template1 > template0 > postgres > omega 2 > (4 rows) > > I dislike this - we know, so the name must be quoted in file (without it, > the file was incorrect). And if you need quotes, there is a function > quote_ident. If we use quotes elsewhere, then it should be ok, bot not now. > Please, remove it. More, it is not necessary, when you use a "keyword" > columns.
Thanks. The special handling of quotes for pg_hba_conf is not required. I will keep the behaviour similar to the other catalog tables. I will remove the quote support in the next version.