Обсуждение: [pgAdmin4][Patch]: Fix RM #3191 : Debug option is not working
Hi,
Please find the attached patch to fix #3191 : Debug option is not working.
Issues fixed:
1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.
2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type
3. The Reverse engineering SQL is not correct for Packages' functions/procedures
4. In case of INOUT argument, debugger asks for mendatory input which should not.
Thanks,
Khushboo
Вложения
Please find the attached updated patch with PEP8 fixes.
On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.Thanks,Khushboo
Вложения
On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Please find the attached updated patch with PEP8 fixes.On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.
Not sure if this is a result of your patch or not, but I'm seeing a failure if I debug a procedure, then click run to debug it again in the same debugger panel. The parameters dialogue opens, but nothing happens when it's closed. Can you check please?
This is on PG 11:
-- PROCEDURE: public.dummy_proc(integer)
-- DROP PROCEDURE public.dummy_proc(integer);
CREATE OR REPLACE PROCEDURE public.dummy_proc(
id integer)
LANGUAGE 'plpgsql'
AS $BODY$BEGIN
raise notice 'id is %', id;
END;$BODY$;
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Hi Dave,
On Wed, Jul 11, 2018 at 4:15 PM, Dave Page <dpage@pgadmin.org> wrote:
On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Please find the attached updated patch with PEP8 fixes.On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.HiNot sure if this is a result of your patch or not, but I'm seeing a failure if I debug a procedure, then click run to debug it again in the same debugger panel. The parameters dialogue opens, but nothing happens when it's closed. Can you check please?
This is working fine for me with/without my patch on PG 11.
This is on PG 11:-- PROCEDURE: public.dummy_proc(integer)-- DROP PROCEDURE public.dummy_proc(integer);CREATE OR REPLACE PROCEDURE public.dummy_proc(id integer)LANGUAGE 'plpgsql'AS $BODY$BEGINraise notice 'id is %', id;END;$BODY$;--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Thanks,
Khushboo
Hi
--
On Thu, Jul 12, 2018 at 6:13 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi Dave,On Wed, Jul 11, 2018 at 4:15 PM, Dave Page <dpage@pgadmin.org> wrote:On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Please find the attached updated patch with PEP8 fixes.On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.HiNot sure if this is a result of your patch or not, but I'm seeing a failure if I debug a procedure, then click run to debug it again in the same debugger panel. The parameters dialogue opens, but nothing happens when it's closed. Can you check please?This is working fine for me with/without my patch on PG 11.
Odd. I've attached a video of the issue so you can see exactly what I'm doing. The server is PG 11b2, running on macOS.
This is on PG 11:-- PROCEDURE: public.dummy_proc(integer)-- DROP PROCEDURE public.dummy_proc(integer);CREATE OR REPLACE PROCEDURE public.dummy_proc(id integer)LANGUAGE 'plpgsql'AS $BODY$BEGINraise notice 'id is %', id;END;$BODY$;--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL CompanyThanks,Khushboo
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Вложения
Hi,
Please find the attached updated patch.
On Thu, Jul 12, 2018 at 2:55 PM, Dave Page <dpage@pgadmin.org> wrote:
HiOn Thu, Jul 12, 2018 at 6:13 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi Dave,On Wed, Jul 11, 2018 at 4:15 PM, Dave Page <dpage@pgadmin.org> wrote:On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Please find the attached updated patch with PEP8 fixes.On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.HiNot sure if this is a result of your patch or not, but I'm seeing a failure if I debug a procedure, then click run to debug it again in the same debugger panel. The parameters dialogue opens, but nothing happens when it's closed. Can you check please?This is working fine for me with/without my patch on PG 11.Odd. I've attached a video of the issue so you can see exactly what I'm doing. The server is PG 11b2, running on macOS.
Fixed.
Thanks,
Khushboo
This is on PG 11:-- PROCEDURE: public.dummy_proc(integer)-- DROP PROCEDURE public.dummy_proc(integer);CREATE OR REPLACE PROCEDURE public.dummy_proc(id integer)LANGUAGE 'plpgsql'AS $BODY$BEGINraise notice 'id is %', id;END;$BODY$;--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL CompanyThanks,Khushboo--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Вложения
Thanks, patch applied.
On Mon, Jul 16, 2018 at 10:26 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,Please find the attached updated patch.On Thu, Jul 12, 2018 at 2:55 PM, Dave Page <dpage@pgadmin.org> wrote:HiOn Thu, Jul 12, 2018 at 6:13 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi Dave,On Wed, Jul 11, 2018 at 4:15 PM, Dave Page <dpage@pgadmin.org> wrote:On Wed, Jul 11, 2018 at 8:03 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Please find the attached updated patch with PEP8 fixes.On Wed, Jul 11, 2018 at 11:18 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote: Hi,Please find the attached patch to fix #3191 : Debug option is not working.Issues fixed:1. EPAS packages' function/procedure does not honour INOUT arguments, it converts INOUT to OUT.2. Packages' functions and procedures are not getting listed in their respected nodes in some scenarios like procedure having INOUT argument and function with void return type3. The Reverse engineering SQL is not correct for Packages' functions/procedures4. In case of INOUT argument, debugger asks for mendatory input which should not.HiNot sure if this is a result of your patch or not, but I'm seeing a failure if I debug a procedure, then click run to debug it again in the same debugger panel. The parameters dialogue opens, but nothing happens when it's closed. Can you check please?This is working fine for me with/without my patch on PG 11.Odd. I've attached a video of the issue so you can see exactly what I'm doing. The server is PG 11b2, running on macOS.Fixed.Thanks,KhushbooThis is on PG 11:-- PROCEDURE: public.dummy_proc(integer)-- DROP PROCEDURE public.dummy_proc(integer);CREATE OR REPLACE PROCEDURE public.dummy_proc(id integer)LANGUAGE 'plpgsql'AS $BODY$BEGINraise notice 'id is %', id;END;$BODY$;--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL CompanyThanks,Khushboo--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company