On Tue, May 5, 2015 at 6:48 AM, Peter Eisentraut <peter_e@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