Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not present when seeing generated SQL from a table

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM4037] COMMENTS from inherited fields are not present when seeing generated SQL from a table
Date: 2019-03-28 16:24:28
Message-ID: CA+OCxoxsDz04Fn-6P4DPDWZBBu-b5V1tnUJ23AGNj4fXbbBAbA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, applied.

On Thu, Mar 28, 2019 at 2:00 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)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(dot)toshniwal(at)enterprisedb(dot)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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2019-03-28 17:13:50 pgAdmin 4 commit: Add support for IDENTITY columns. Fixes #4030
Previous Message Khushboo Vashi 2019-03-28 13:18:12 [pgAdmin4][Patch] - RM 4082 - Download CSV error displayed for procedure and functions with create script