Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not presentwhen seeing generated SQL from a table - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not presentwhen seeing generated SQL from a table
Date
Msg-id CAM9w-_nPNBNNqCrA_Pj1JVvDjYGdwVE+rXBEf4nv8nNuxQXgdA@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not presentwhen seeing generated SQL from a table  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not presentwhen seeing generated SQL from a table
List pgadmin-hackers
Hi Hackers,

I missed few SQLs for partition tables, gpdb, ppas and other pg versions :(
Attached is the patch to include those.

Kindly review.

On Thu, Mar 28, 2019 at 9:54 PM Dave Page <dpage@pgadmin.org> wrote:
Thanks, applied.

On Thu, Mar 28, 2019 at 2:00 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Just to mention, the patch also fixes RM2627, RM4058.

On Wed, Mar 27, 2019 at 6:25 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch which fixes the issue where comments added in child table on inherited columns where not showing in reverse engineered SQL.

I have also made changes to show column details of inherited columns of table, as well as for "of type" tables similar to pgAdmin3. eg-
CREATE TABLE public.inhactortest
(
-- Inherited from table public.actor: actor_id integer NOT NULL DEFAULT nextval('actor_actor_id_seq'::regclass),
-- Inherited from table public.actor: first_name character varying(45) COLLATE pg_catalog."default" NOT NULL,
-- Inherited from table public.actor: last_name character varying(45) COLLATE pg_catalog."default" NOT NULL,
-- Inherited from table public.test_table: col1 integer,
-- Inherited from table public.test_table: col2 "char",
-- Inherited from table public.test_table: col3 character varying COLLATE pg_catalog."default",
 owncol integer
)
    INHERITS (public.actor, public.test_table)
WITH (
    OIDS = FALSE
)
TABLESPACE pg_default;

ALTER TABLE public.inhactortest
    OWNER to postgres;

COMMENT ON COLUMN public.inhactortest.actor_id
    IS 'in child c';
-- 

Kindly review.

Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"


--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"
Attachment

pgadmin-hackers by date:

Previous
From: Khushboo Vashi
Date:
Subject: [pgAdmin4][Patch] - Minor patch to remove javascript debugger
Next
From: Usman Muzaffar
Date:
Subject: Feature Tests query tool: unnecessary print statement removal