Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1h6fdM-0002N4-Ou for pgsql-committers@arkaria.postgresql.org; Wed, 20 Mar 2019 18:08:12 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1h6fdK-0004ZZ-38 for pgsql-committers@arkaria.postgresql.org; Wed, 20 Mar 2019 18:08:10 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1h6fdJ-0004TO-SR for pgsql-committers@lists.postgresql.org; Wed, 20 Mar 2019 18:08:09 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1h6fdG-000499-PN for pgsql-committers@lists.postgresql.org; Wed, 20 Mar 2019 18:08:08 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id x2KI83NX014000; Wed, 20 Mar 2019 14:08:03 -0400 From: Tom Lane To: Peter Geoghegan cc: pgsql-committers Subject: Re: pgsql: Make heap TID a tiebreaker nbtree index column. In-reply-to: References: Comments: In-reply-to Peter Geoghegan message dated "Wed, 20 Mar 2019 11:00:06 -0700" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <13998.1553105283.1@sss.pgh.pa.us> Date: Wed, 20 Mar 2019 14:08:03 -0400 Message-ID: <13999.1553105283@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Peter Geoghegan writes: > On Wed, Mar 20, 2019 at 10:05 AM Peter Geoghegan wrote: >> Make heap TID a tiebreaker nbtree index column. > I see that this has caused SELinux test failures on rhinoceros (the > ddl test fails). It looks like the output order is affected by the > implementation details of nbtree, likely for some system catalog > index. This is something that I've had to deal with in other places > (with a lot of help from Tom). The diffs all are related to the order of operations in a DROP OWNED BY command, so I think blaming SELinux is just blaming the messenger. This comes down to the point that we didn't do anything to ensure drop order stability in shdepend-driven drops. Maybe we need to be honest about that. Or do you have reason to think that your changes will result in stability in that drop order anyway? If so, why? regards, tom lane