Re: Should we rename amapi.h and amapi.c?

2020-01-02 Thread Andres Freund
Hi, (Moving discussion from [1] to this thread) On 2019-12-28 11:32:26 -0500, Tom Lane wrote: > Andres Freund writes: > > On 2019-12-27 08:20:17 +0900, Michael Paquier wrote: > >> Hm, I am not sure that it is actually that much used, such stuff is > >> very specialized. > > > That's true for

Re: Should we rename amapi.h and amapi.c?

2019-12-24 Thread Michael Paquier
On Tue, Dec 24, 2019 at 02:22:22PM +0100, Fabien COELHO wrote: > The change does not attempt to keep included files in ab order. Should it do > that, or is it fixed later by some reindentation phase? Yeah, it should. Committed after fixing all that stuff. -- Michael signature.asc Description:

Re: Should we rename amapi.h and amapi.c?

2019-12-24 Thread Fabien COELHO
Bonjour Michaƫl, the syscache mainly, so instead the attached patch does the following changes: - amapi.h -> indexam.h - amapi.c -> indexamapi.c. Here we have an equivalent in access/table/ as tableamapi.c. - amvalidate.c -> indexamvalidate.c - amvalidate.h -> indexamvalidate.h - genam.c ->

Re: Should we rename amapi.h and amapi.c?

2019-12-24 Thread Michael Paquier
On Tue, Dec 24, 2019 at 09:32:23AM +0100, Julien Rouhaud wrote: > Looks good to me. There are still references to amapi.c in various > .po files, but those should rather be taken care of with the next > update-po cycle right? Yes, these are updated as part of the translation updates. -- Michael

Re: Should we rename amapi.h and amapi.c?

2019-12-24 Thread Julien Rouhaud
On Tue, Dec 24, 2019 at 3:57 AM Michael Paquier wrote: > > On Mon, Dec 23, 2019 at 12:28:36PM -0800, Ashwin Agrawal wrote: > > I had raised the same earlier and [1] has response from Andres, which was > > "We probably should rename it, but not in 12..." > > > > [1] > >

Re: Should we rename amapi.h and amapi.c?

2019-12-23 Thread Michael Paquier
On Mon, Dec 23, 2019 at 12:28:36PM -0800, Ashwin Agrawal wrote: > I had raised the same earlier and [1] has response from Andres, which was > "We probably should rename it, but not in 12..." > > [1] > https://www.postgresql.org/message-id/20190508215135.4eljnhnle5xp3jwb%40alap3.anarazel.de Okay,

Re: Should we rename amapi.h and amapi.c?

2019-12-23 Thread Ashwin Agrawal
On Sun, Dec 22, 2019 at 9:34 PM Michael Paquier wrote: > Hi all, > > I was working on some stuff for table AMs, and I got to wonder it we > had better rename amapi.h to indexam.h and amapi.c to indexam.c, so as > things are more consistent with table AM. It is a bit annoying to > name the files

Re: Should we rename amapi.h and amapi.c?

2019-12-23 Thread David Fetter
On Mon, Dec 23, 2019 at 02:34:34PM +0900, Michael Paquier wrote: > Hi all, > > I was working on some stuff for table AMs, and I got to wonder it we > had better rename amapi.h to indexam.h and amapi.c to indexam.c, so as > things are more consistent with table AM. It is a bit annoying to > name

Should we rename amapi.h and amapi.c?

2019-12-22 Thread Michael Paquier
Hi all, I was working on some stuff for table AMs, and I got to wonder it we had better rename amapi.h to indexam.h and amapi.c to indexam.c, so as things are more consistent with table AM. It is a bit annoying to name the files dedicated to index AMs with what looks like now a too generic name.