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

From Daniel Gustafsson
Subject Re: Remove obsolete pg_attrdef.adsrc column
Date
Msg-id 3A344FC1-C695-478A-BD1A-6ADFA50C6227@yesql.se
Whole thread Raw
In response to Re: Remove obsolete pg_attrdef.adsrc column  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Responses Re: Remove obsolete pg_attrdef.adsrc column  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
> On 27 Oct 2018, at 12:57, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote:
>
> 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.

No objections from me.

> Updated patches attached.

+1, applies and works as intended.

cheers ./daniel


pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Conflicting option checking in pg_restore
Next
From: Tomas Vondra
Date:
Subject: Re: vacuum fails with "could not open statistics file" "Device orresource busy"