From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Kouhei Sutou <kou(at)clear-code(dot)com> |
Cc: | pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: BUG #13840: pg_dump generates unloadable SQL when third party string type index option is used |
Date: | 2016-01-02 17:50:54 |
Message-ID: | 21062.1451757054@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
Kouhei Sutou <kou(at)clear-code(dot)com> writes:
> Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Pushed with some adjustments (notably, I thought the quoting rule was
>> too complicated and not necessarily 100% correct).
> Thanks for merging my patch and backporting to released
> series.
I looked into pg_dump and realized that this fixes only a few of the
problems in this area. While pg_dump does rely on ruleutils.c to
print reloptions of simple indexes, it does not do that for reloptions
of tables or views, nor for reloptions of indexes that are constraints.
So eventually that's going to bite us on the rear, though I'm not sure
if we have a live problem today.
One could imagine exporting flatten_reloptions via a separate SQL
function, but that could only exist in future releases, so I'm afraid
we're going to have to duplicate the functionality inside pg_dump.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | rob stone | 2016-01-02 17:56:51 | Re: BUG #13841: Unable to set autocommit in psql |
Previous Message | Kouhei Sutou | 2016-01-02 14:43:21 | Re: BUG #13840: pg_dump generates unloadable SQL when third party string type index option is used |