Re: patch: option --if-exists for pg_dump

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: patch: option --if-exists for pg_dump
Дата
Msg-id CAFj8pRC_n+aLU4M51wAUjNJGbUz1+gMuyyC7gAJ0N=NAXD6RqA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: patch: option --if-exists for pg_dump  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
Ответы Re: patch: option --if-exists for pg_dump  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
Список pgsql-hackers
Hello

patch with updated comment

regards

Pavel



2014-01-30 Jeevan Chalke <jeevan.chalke@enterprisedb.com>:
Hi Pavel,

Ok. Good.
Sorry I didn't update my sources. Done now. Thanks
 
 

Also, I didn't quite understand these lines of comments:

                        /*
                         * Descriptor string (te-desc) should not be same as object
                         * specifier for DROP STATEMENT. The DROP DEFAULT has not
                         * IF EXISTS clause - has not sense.
                         */

Will you please rephrase ?

I can try it - .

A content of te->desc is usually substring of DROP STATEMENT with one related exception - CONSTRAINT.
Independent to previous sentence - ALTER TABLE ALTER COLUMN DROP DEFAULT doesn't support IF EXISTS - and therefore it should not be injected.

is it ok?

Fine with me.

Thanks

--
Jeevan B Chalke
Principal Software Engineer, Product Development
EnterpriseDB Corporation
The Enterprise PostgreSQL Company


Вложения

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

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: Add min and max execute statement time in pg_stat_statement
Следующее
От: Rushabh Lathia
Дата:
Сообщение: Re: pg_upgrade: make the locale comparison more tolerating