Re: [pgadmin-hackers] Fixed PPAS trigger column view condition

2014-10-13 Thread Ashesh Vashi
On Mon, Oct 13, 2014 at 2:35 PM, Dave Page wrote: > OK, thanks. Ashesh, please review and commit - thanks! > Sure. -- Thanks & Regards, Ashesh Vashi EnterpriseDB INDIA: Enterprise PostgreSQL Company *http://www.linkedin.com/in/asheshvashi*

Re: [pgadmin-hackers] Fixed PPAS trigger column view condition

2014-10-13 Thread Dave Page
OK, thanks. Ashesh, please review and commit - thanks! On Mon, Oct 13, 2014 at 10:04 AM, Sanket Mehta wrote: > Hi Dave, > > per trigger column is available in PPAS since 8.5 (9.0) version. > It seems that it was wrongly assume by "Guillaume Lelarge" who has added > this check in pgadmin that is s

Re: [pgadmin-hackers] Fixed PPAS trigger column view condition

2014-10-13 Thread Sanket Mehta
Hi Dave, per trigger column is available in PPAS since 8.5 (9.0) version. It seems that it was wrongly assume by "Guillaume Lelarge" who has added this check in pgadmin that is should not support per column trigger for PPAS. So the current patch is working for PG and PPAS both. Please let me know

Re: [pgadmin-hackers] Fixed PPAS trigger column view condition

2014-10-10 Thread Dave Page
On Fri, Oct 10, 2014 at 1:35 PM, Sanket Mehta wrote: > Hi Dave, > > There is one issue with pgadmin while viewing the trigger definition created > for PPAS server as below: > > CREATE OR REPLACE TRIGGER tr_a > BEFORE UPDATE OF cl1 ON a FOR EACH ROW > begin > if(:new.cl1 < 0) then > insert into