On Tue, Jun 13, 2017 at 5:22 AM, Amit Langote <langote_amit...@lab.ntt.co.jp> wrote: > Yeah, I was thinking the same while writing the patch posted on the thread > "A bug in mapping attributes in ATExecAttachPartition()" . That patch > adds the break you mention in 2, but didn't do anything about point 1. > > In any case, +1 to your patch. I'll rebase if someone decides to commit > it first.
If the patch I posted in https://www.postgresql.org/message-id/CA%2BTgmoYmW9VwCWDpe7eXUxeKmAKOxmg8itgFkB5UTQTq4SnTjQ%40mail.gmail.com gets committed, all of this code will be gone entirely, so this will be moot. If we decide to repair the existing broken logic rather than ripping it out entirely then this is probably a good idea, but I hope that's not what happens. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (firstname.lastname@example.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers