On Tue, May 5, 2015 at 6:48 AM, Peter Eisentraut <pete...@gmx.net> wrote: > On 5/1/15 12:33 PM, Andres Freund wrote: >> On 2015-04-08 19:19:29 +0100, Greg Stark wrote: >>> I'm not sure what the best way to handle the hand-off from patch >>> contribution to reviewer/committer. If I start tweaking things then >>> you send in a new version it's actually more work to resolve the >>> conflicts. I think at this point it's easiest if I just take it from >>> here. >> >> Are you intending to commit this? > > It still looks quite dubious to me. > > The more I test this, the more fond I grow of the idea of having this > information available in SQL. But I'm also growing more perplexed by > how this the file is mapped to a table. It just isn't a good match. > > For instance: What is keyword_databases? Why is it an array? Same for > keyword_users. How can I know whether a given database or user matches > a keyword? What is compare_method? (Should perhaps be > keyword_address?) Why is compare method set to "mask" when a hostname > is set? (Column order is also a bit confusing here.) I'd also like > options to be jsonb instead of a text array.
Thanks for your suggestion. I am not sure how to use jsonb here, i will study the same and provide a patch for the next version. Regards, Hari Babu Fujitsu Australia -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers