Re: pg_dump bug for extension owned tables

Поиск
Список
Период
Сортировка
От Andrew Dunstan
Тема Re: pg_dump bug for extension owned tables
Дата
Msg-id a86f1cb7-e2eb-9522-1d3e-f8768ad87ef0@2ndQuadrant.com
обсуждение исходный текст
Ответ на Re: pg_dump bug for extension owned tables  (Fabrízio de Royes Mello <fabriziomello@gmail.com>)
Ответы Re: pg_dump bug for extension owned tables  (Fabrízio de Royes Mello <fabriziomello@gmail.com>)
Список pgsql-hackers
On 6/26/20 2:10 PM, Fabrízio de Royes Mello wrote:
>
> On Fri, Jun 26, 2020 at 11:55 AM Fabrízio de Royes Mello
> <fabriziomello@gmail.com <mailto:fabriziomello@gmail.com>> wrote:
> >
> >
> > On Fri, Jun 26, 2020 at 11:24 AM Andrew Dunstan
> <andrew.dunstan@2ndquadrant.com
> <mailto:andrew.dunstan@2ndquadrant.com>> wrote:
> > >
> > >
> > > On 6/26/20 9:57 AM, Andrew Dunstan wrote:
> > > > It appears that for extension owned tables tbinfo.attgenerated isn't
> > > > being properly populated, so line 2050 in REL_12_STABLE, which
> is line
> > > > 2109 in git tip, is failing.
> > > >
> > > >
> > >
> > > Should have mentioned this is in src/bin/pg_dump/pg_dump.c
> > >
> >
> > Having a look on it.
> >
>
> Seems when qualify the schemaname the the "tbinfo->interesting" field
> is not setted for extensions objects, so the getTableAttrs can't fill
> the attgenerated field properly.
>
> I'm not 100% sure it's the correct way but the attached patch works
> for me and all tests passed. Maybe we should add more TAP tests?
>
>


I just tried this patch out on master, with the test case I gave
upthread. It's not working, still getting a segfault.


Will look closer.


cheers


andrew


-- 
Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services




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

Предыдущее
От: Justin Pryzby
Дата:
Сообщение: Re: proposal: possibility to read dumped table's name from file
Следующее
От: Tomas Vondra
Дата:
Сообщение: Re: Default setting for enable_hashagg_disk