Re: Exclude pg_largeobject form pg_dump

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: Exclude pg_largeobject form pg_dump
Дата
Msg-id CAECtzeXExEOvuf=XLpdPnyNTK5cOhxmKt8K28HYuynCzsMKQdg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Exclude pg_largeobject form pg_dump  (amul sul <sulamul@gmail.com>)
Ответы Re: Exclude pg_largeobject form pg_dump  (Guillaume Lelarge <guillaume@lelarge.info>)
Список pgsql-general
Hi Amul,

2016-11-04 7:52 GMT+01:00 amul sul <sulamul@gmail.com>:
Hi Guillaume,

I found following issues with this patch, sorry missed in previous post:


You don't have to be sorry for me doing shitty things :)
 
#1 :
 43 @@ -392,6 +393,10 @@ main(int argc, char **argv)
 44                 dopt.outputBlobs = true;
 45                 break;
 46
 47 +           case 'B':           /* Don't dump blobs */
 48 +               dopt.include_everything = false;
 49 +               break;
 50 +

Touching dopt.include_everything flag does not seems to be a good idea
for '--no-blobs' option, our intension is to exclude blob only, but
this excluds other dump too (e.g COMMENT ON DATABASE, CREATE
EXTENSION, COMMENT ON EXTENSION, .., etc)  that what we don't want,
right?


Agreed. I was afraid of that, but for some reason, didn't find that. I'll fix this.
 
#2 :
We should add note for default behaviour if --no-blobs & --blobs both
are specified.


Right. I don't know how I will handle this, but you're right that the behaviour should be specified. I'll also fix this.

I'll try to work on this today but as I'm in pgconf.eu 2016, it may be only for tomorrow.

Thank you.


--

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

Предыдущее
От: amul sul
Дата:
Сообщение: Re: Exclude pg_largeobject form pg_dump
Следующее
От: Nhan Nguyen
Дата:
Сообщение: High load average every 105 minutes