Postgres Pro
Downloads
Home   >   mailing lists

Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator
Date
Msg-id CA+TgmoaA7FhgWDZ6Nu42iLhSFuso2dxUPA3x6rd+0EniS42JYg@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator  (Tomas Vondra <tomas.vondra@2ndquadrant.com>)
Responses Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator
List pgsql-hackers
Tree view 
On Tue, Mar 22, 2016 at 9:25 PM, Tomas Vondra
<tomas.vondra@2ndquadrant.com> wrote:
> OK, the new code seems more comprehensible to me.

I did not find this version very clear.  It wasn't consistent about
using ObjectIdGetDatum() where needed, but the bigger problem was that
I found the logic unnecessarily convoluted.  I rewrote it - I believe
more straightforwardly - as attached.  How does this look?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Attachment

pgsql-hackers by date:

Previous
From: Jim Nasby
Date:
Subject: Re: Relax requirement for INTO with SELECT in pl/pgsql
Next
From: Aleksander Alekseev
Date:
Subject: Re: Patch: fix lock contention for HASHHDR.mutex