On Mon, Mar 30, 2015 at 4:34 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote: > Hi > > I checked this patch. I like the functionality and behave.
Thanks for the review.
Here I attached updated patch with the following changes.
1. Addition of two new keyword columns
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"
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"
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.