Re: [pgAdmin][RM5601] : [Schema Diff] Add RLS policy support in schema diff

2020-07-01 Thread Akshay Joshi
Thanks, patch applied. On Wed, Jul 1, 2020 at 11:04 AM Pradip Parkale < pradip.park...@enterprisedb.com> wrote: > Hi Akshay, > > Please find the attached patch for following fixes, > >- RM5631 - [RLS Policy] can't execute an empty query. >- In case of more than one policy, all the headers

Re: [pgAdmin][RM5601] : [Schema Diff] Add RLS policy support in schema diff

2020-06-30 Thread Pradip Parkale
Hi Akshay, Please find the attached patch for following fixes, - RM5631 - [RLS Policy] can't execute an empty query. - In case of more than one policy, all the headers are showing as a comment on top of policy DDL in schema diff. - Fixed - If the policy event is different for source a

Re: [pgAdmin][RM5601] : [Schema Diff] Add RLS policy support in schema diff

2020-06-26 Thread Akshay Joshi
Hi Pradip Following are the review comments: - For function arguments please make sure the right sequence is *gid, sid, did, scid, ... *All the parameters are not compulsory but the parameters should be in the mentioned order for consistency. Change it to all the functions of RLS Pol

[pgAdmin][RM5601] : [Schema Diff] Add RLS policy support in schema diff

2020-06-25 Thread Pradip Parkale
Hi Hackers, Please find the attached patch for RLS policy support in schema diff. -- Thanks & Regards, Pradip Parkale QMG, EnterpriseDB Corporation RM-5601.patch Description: Binary data