From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>
Cc: Jesper Pedersen <jesper.pedersen@redhat.com>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: David Rowley <david.rowley@2ndquadrant.com>
Cc: Simon Riggs <simon@2ndquadrant.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Cc: Maksim Milyutin <milyutinma@gmail.com>
Subject: Re: [HACKERS] Proposal: Local indexes for partitioned table
Date: Thu, 4 Jan 2018 11:44:15 -0500
Message-ID: <239c2d51-2762-4e03-339b-d36f2f69edca@2ndquadrant.com> (raw)
In-Reply-To: <20171229175930.3aew7lzwd5w6m2x6@alvherre.pgsql>
References: <20171229175930.3aew7lzwd5w6m2x6@alvherre.pgsql>
On 12/29/17 12:59, Alvaro Herrera wrote:
>> Maybe we need a new "auto internal" deptype with a mix of semantics of
>> the other two deptypes. It seems a bit ugly and I'm not sure it'd work
>> either ... I'll try to code it tomorrow.
>
> This seems to work pretty well, much to my surprise. I was a bit scared
> of adding a new deptype, but actually the only affected code is
> findDependentObjects() and the semantics of the new type is a subset of
> the existing DEPTYPE_INTERNAL, so I think it's okay. I need to fill in
> its description in docs and comments though -- I left it out because the
> real difference between INTERNAL and INTERNAL_AUTO is not something that
> is covered by the existing description of INTERNAL, so maybe I'll need
> to expand that one.
So this patch appears to implement what was agreed upon earlier in the
thread. Documentation and tests seem pretty comprehensive.
The new dependency type obviously needs to be explained in detail.
CompareIndexInfo() doesn't compare indexes' operator classes and collations.
I'm not sure why this feature of automatically picking up matching
indexes even exists. Is it for some specific workflows or upgrade
scenarios? It's kind of a surprising feature in a way.
The catalog representations of partitioned tables and partitioned
indexes are completely different, which may or may not be desirable.
As mentioned elsewhere already, the tests fail because \di shows the
owner, which can vary between sites.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: peter.eisentraut@2ndquadrant.com, alvherre@alvh.no-ip.org, Langote_Amit_f8@lab.ntt.co.jp, jesper.pedersen@redhat.com, robertmhaas@gmail.com, david.rowley@2ndquadrant.com, simon@2ndquadrant.com, milyutinma@gmail.com
Subject: Re: [HACKERS] Proposal: Local indexes for partitioned table
In-Reply-To: <239c2d51-2762-4e03-339b-d36f2f69edca@2ndquadrant.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox