Re: Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-11-11 Thread Greg Sabino Mullane
Thanks so much: the Bucardo PR has been merged in.

Re: Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-11-10 Thread Michael Paquier
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 signature.asc Description: PGP signature

Re: Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-11-10 Thread Andreas Karlsson
On 11/3/20 10:22 AM, Magnus Hagander wrote: On Thu, Sep 24, 2020 at 5:17 AM Michael Paquier 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:

Re: Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-11-03 Thread Magnus Hagander
On Thu, Sep 24, 2020 at 5:17 AM Michael Paquier 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 > >

Re: Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-09-23 Thread Michael Paquier
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

Prefer TG_TABLE_NAME over TG_RELNAME in tests

2020-09-23 Thread Daniel Gustafsson
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