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-29 14:44:38 |
Message-ID: | CA+OCxowQH-gags=RLJkS6QVtXSKAY=2V_vi2CQXAXSU3QAbVCw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks - applied.
On Fri, Mar 29, 2019 at 7:42 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> 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(at)pgadmin(dot)org> wrote:
>
>> 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
>>
>
>
> --
> 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
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2019-03-29 14:44:46 | pgAdmin 4 commit: Add some SQL updates to handle inherited column marke |
Previous Message | Dave Page | 2019-03-29 14:41:01 | pgAdmin 4 commit: Remove a debug statement. |