From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Teodor Sigaev <teodor(at)sigaev(dot)ru>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: CUBE seems a bit confused about ORDER BY |
Date: | 2018-01-11 15:55:21 |
Message-ID: | 14120.1515686121@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> Tom Lane wrote:
>> Was there any real discussion of whether we could get away with that
>> in the back branches? My opinion is no. It's not even clear to me
>> that this is acceptable in HEAD --- isn't it going to create huge
>> problems for pg_upgrade?
> This was discussed upthread and the solution found was "objects need to
> be rebuilt, indexes need to be reindexed". The point of Alexander's
> query was to find affected objects that need such treatment. Teodor
> explicitly disregarded any change in pg_upgrade because the database
> you're upgrading *from* is supposed to have gotten indexes reindexed,
> etc.
I don't think that is really going to be acceptable. People do not like
minor version updates that break their databases. If we start doing
that we're going to find people refusing to apply minor updates, which
is not a place we want to be.
What we've done in the past for comparable situations is to make the
change in a new major version and teach pg_upgrade to detect and report
the need for changes --- in this case, it might do something like create
a script of REINDEX commands for the affected indexes. See e.g.
old_9_6_invalidate_hash_indexes().
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Jeremy Finzel | 2018-01-11 15:56:32 | Identifying ALTER TABLE "sub-command" |
Previous Message | Arthur Zakirov | 2018-01-11 15:53:49 | Re: [HACKERS] PoC: custom signal handler for extensions |