public inbox for [email protected]  
help / color / mirror / Atom feed
From: Nathan Bossart <[email protected]>
To: Bharath Rupireddy <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Do away with a few backwards compatibility macros
Date: Mon, 27 Nov 2023 13:14:39 -0600
Message-ID: <20231127191439.GA47360@nathanxps13> (raw)
In-Reply-To: <CALj2ACV4dRfCqpmY+kgYxDuqAAqV_5iqFdaWvnuuUn79JXLv4A@mail.gmail.com>
References: <CALj2ACVeO58JM5tK2Qa8QC-=kC8sdkJOTd4BFU=K8zs4gGYpjQ@mail.gmail.com>
	<20231116154622.GA2881373@nathanxps13>
	<20231121045840.GD3521465@nathanxps13>
	<[email protected]>
	<20231121155203.GA3566722@nathanxps13>
	<CALj2ACV4dRfCqpmY+kgYxDuqAAqV_5iqFdaWvnuuUn79JXLv4A@mail.gmail.com>

On Mon, Nov 27, 2023 at 04:29:18PM +0530, Bharath Rupireddy wrote:
> I think it's easy to miss/enforce a documented policy. IMV, moving
> towards pg_attribute_deprecated as Alvaro Herrera said in the other
> thread https://www.postgresql.org/message-id/202311141920.edtj56saukiv%40alvherre.pgsql
> can help. Authors then can declare the variables and functions as
> deprecated so that the code compilation with
> -Wno-deprecated-declarations can help track all such deprecated code.

I'm +1 for adding pg_attribute_deprecated once we have something to use it
for.

> Having said that, I'm all +1 if the v1 patch proposed in this thread gets in.

Committed.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com






view thread (7+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Do away with a few backwards compatibility macros
  In-Reply-To: <20231127191439.GA47360@nathanxps13>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox