Re: [PATCH] Tables node (pgAdmin4)

Поиск
Список
Период
Сортировка
От Thom Brown
Тема Re: [PATCH] Tables node (pgAdmin4)
Дата
Msg-id CAA-aLv46ECN_NZJ=Uercv7mvE+s=vceXqZsb32Qfnh_Oq6OhcA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Tables node (pgAdmin4)  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
Ответы Re: [PATCH] Tables node (pgAdmin4)  (Thom Brown <thom@linux.com>)
Список pgadmin-hackers
On 24 May 2016 at 19:09, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Mon, May 23, 2016 at 6:35 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:

Hi,

PFA patch, which will fixes below mentioned issues,
Committed.

In the Create Table dialog, on the Advanced tab, the "Of type" drop-down lists tables and composite types, but those are supposed to just be composite types.

Also, when using OF in CREATE TABLE, LIKE is no longer valid, so LIKE should be disabled when using OF.  Also, there should probably be a way of setting options for the columns taken from the composite type.  For example:

CREATE TYPE inventory AS (product_id bigint, product_name text, weight numeric);

CREATE TABLE stock OF inventory (
    PRIMARY KEY (product_id),
    weight WITH OPTIONS DEFAULT 0
);

There's currently no way of doing this (neither the primary key, nor the default value for any columns).  It should probably automatically populate the columns from the composite type on the columns tab.

Also, could the generated SQL have an empty line between each statement?

Thanks

Thom

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Ashesh Vashi
Дата:
Сообщение: Re: server and database node type check in is_connected check [pgadmin4]
Следующее
От: Thom Brown
Дата:
Сообщение: Re: [PATCH] Tables node (pgAdmin4)