Re: Remove obsolete pg_attrdef.adsrc column - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Remove obsolete pg_attrdef.adsrc column
Date
Msg-id fc63c5be-94b5-1e2b-07a4-aa52be7da545@2ndquadrant.com
Whole thread Raw
In response to Re: Remove obsolete pg_attrdef.adsrc column  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Remove obsolete pg_attrdef.adsrc column  (Daniel Gustafsson <daniel@yesql.se>)
List pgsql-hackers
On 23/10/2018 19:48, Daniel Gustafsson wrote:
>> On 23 Oct 2018, at 15:17, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote:
>>
>> I propose the attached patch to remove the long-unused catalog column
>> pg_attrdef.adsrc.
> 
> +1, I ran into a bug in an app as recently as today where adsrc was used
> instead of pg_get_expr().
> 
> Patch looks good.  I probably would’ve opted for mentioning how to get a human
> readable version on the page, along the lines of the attached version,

Agreed.  I have integrated your suggestion.

Also, let's go nuts and remove pg_constraint.consrc as well.

Updated patches attached.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Resetting PGPROC atomics in ProcessInit()
Next
From: Antonin Houska
Date:
Subject: Re: Comment fix and question about dshash.c