Thread: Prefer TG_TABLE_NAME over TG_RELNAME in tests

TG_RELNAME was marked deprecated in commit 3a9ae3d2068 some 14 years ago, but
we still use it in the triggers test suite (test added in 59b4cef1eb74a a year
before deprecation).  Seems about time to move over to TG_TABLE_NAME ourselves,
as TG_RELNAME is still covered by the test added in the deprecation commit.

cheers ./daniel

On Wed, Sep 23, 2020 at 12:07:14PM +0200, Daniel Gustafsson wrote:
> TG_RELNAME was marked deprecated in commit 3a9ae3d2068 some 14 years ago, but
> we still use it in the triggers test suite (test added in 59b4cef1eb74a a year
> before deprecation).  Seems about time to move over to TG_TABLE_NAME ourselves,
> as TG_RELNAME is still covered by the test added in the deprecation commit.

No objections from here to remove that from the core tests.  It is
worth noting that Debian Code Search hints that this is used in some
extensions:
https://codesearch.debian.net/search?q=TG_RELNAME&literal=1

These are pgformatter, bucardo, sql-ledger, ledgersmb and davical.
--
Michael

On Thu, Sep 24, 2020 at 5:17 AM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Wed, Sep 23, 2020 at 12:07:14PM +0200, Daniel Gustafsson wrote:
> > TG_RELNAME was marked deprecated in commit 3a9ae3d2068 some 14 years ago, but
> > we still use it in the triggers test suite (test added in 59b4cef1eb74a a year
> > before deprecation).  Seems about time to move over to TG_TABLE_NAME ourselves,
> > as TG_RELNAME is still covered by the test added in the deprecation commit.
>
> No objections from here to remove that from the core tests.  It is
> worth noting that Debian Code Search hints that this is used in some
> extensions:
> https://codesearch.debian.net/search?q=TG_RELNAME&literal=1
>
> These are pgformatter, bucardo, sql-ledger, ledgersmb and davical.

That's interesting, but I think irrelevant to this patch in itself of
course. But it might be worth reaching out to some of those projects
and notifying them they're using the deprecated ones..

Thus, pushed. Thanks!

-- 
 Magnus Hagander
 Me: https://www.hagander.net/
 Work: https://www.redpill-linpro.com/

On 11/3/20 10:22 AM, Magnus Hagander wrote:
> On Thu, Sep 24, 2020 at 5:17 AM Michael Paquier <michael@paquier.xyz> wrote:
>> No objections from here to remove that from the core tests.  It is
>> worth noting that Debian Code Search hints that this is used in some
>> extensions:
>> https://codesearch.debian.net/search?q=TG_RELNAME&literal=1
>>
>> These are pgformatter, bucardo, sql-ledger, ledgersmb and davical.
> 
> That's interesting, but I think irrelevant to this patch in itself of
> course. But it might be worth reaching out to some of those projects
> and notifying them they're using the deprecated ones..

I submitted patches to pgformatter, bucardo and ledgersmb. Both davical 
and sql-ledger only seems to have them in old upgrade scripts.

Andreas

On Tue, Nov 10, 2020 at 09:50:15AM +0100, Andreas Karlsson wrote:
> I submitted patches to pgformatter, bucardo and ledgersmb. Both davical and
> sql-ledger only seems to have them in old upgrade scripts.

Thanks, Andreas!
--
Michael