Re: Inconsistency between table am callback and table function names

2019-05-24 Thread Ashwin Agrawal
On Thu, May 23, 2019 at 4:32 PM Andres Freund wrote: > Hi, > > On 2019-05-14 12:11:46 -0700, Ashwin Agrawal wrote: > > Thank you. Please find the patch to rename the agreed functions. It would > > be good to make all consistent instead of applying exception to three > > functions but seems no con

Re: Inconsistency between table am callback and table function names

2019-05-23 Thread Andres Freund
Hi, On 2019-05-14 12:11:46 -0700, Ashwin Agrawal wrote: > Thank you. Please find the patch to rename the agreed functions. It would > be good to make all consistent instead of applying exception to three > functions but seems no consensus on it. Given table_ api are new, we could > modify them lea

Re: Inconsistency between table am callback and table function names

2019-05-14 Thread Andres Freund
Hi, On 2019-05-14 16:27:47 -0400, Alvaro Herrera wrote: > On 2019-May-14, Ashwin Agrawal wrote: > > > Thank you. Please find the patch to rename the agreed functions. It would > > be good to make all consistent instead of applying exception to three > > functions but seems no consensus on it. Giv

Re: Inconsistency between table am callback and table function names

2019-05-14 Thread Alvaro Herrera
On 2019-May-14, Ashwin Agrawal wrote: > Thank you. Please find the patch to rename the agreed functions. It would > be good to make all consistent instead of applying exception to three > functions but seems no consensus on it. Given table_ api are new, we could > modify them leaving systable_ one

Re: Inconsistency between table am callback and table function names

2019-05-14 Thread Ashwin Agrawal
On Wed, May 8, 2019 at 5:05 PM Ashwin Agrawal wrote: > Not having consistency is the main aspect I wish to bring to > attention. Like for some callback functions the comment is > > /* see table_insert() for reference about parameters */ > void(*tuple_insert) (Relation rel, TupleTa

Re: Inconsistency between table am callback and table function names

2019-05-14 Thread Ashwin Agrawal
On Mon, May 13, 2019 at 12:51 PM Robert Haas wrote: > On Fri, May 10, 2019 at 3:43 PM Ashwin Agrawal > wrote: > > Meant to stick the question mark in that email, somehow missed. Yes > > not planning to spend any time on it if objections. Here is the list > > of renames I wish to perform. > > > >

Re: Inconsistency between table am callback and table function names

2019-05-13 Thread Robert Haas
On Fri, May 10, 2019 at 3:43 PM Ashwin Agrawal wrote: > Meant to stick the question mark in that email, somehow missed. Yes > not planning to spend any time on it if objections. Here is the list > of renames I wish to perform. > > Lets start with low hanging ones. > > table_rescan -> table_scan_re

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Tom Lane
Alvaro Herrera writes: > On 2019-May-10, Andres Freund wrote: >> My personal opinion is that this is more churn than I think is useful to >> tackle after feature freeze, with not sufficient benefits. If others >> chime in, voting to do this, I'm OK with doing that, but otherwise I >> think there'

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Andres Freund
Hi, On 2019-05-10 16:18:32 -0400, Alvaro Herrera wrote: > On 2019-May-10, Andres Freund wrote: > > > My personal opinion is that this is more churn than I think is useful to > > tackle after feature freeze, with not sufficient benefits. If others > > chime in, voting to do this, I'm OK with doin

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Alvaro Herrera
On 2019-May-10, Andres Freund wrote: > My personal opinion is that this is more churn than I think is useful to > tackle after feature freeze, with not sufficient benefits. If others > chime in, voting to do this, I'm OK with doing that, but otherwise I > think there's more important stuff to do.

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Andres Freund
Hi, On 2019-05-10 12:43:06 -0700, Ashwin Agrawal wrote: > On Fri, May 10, 2019 at 10:51 AM Andres Freund wrote: > > > > Hi, > > > > On 2019-05-10 10:43:44 -0700, Ashwin Agrawal wrote: > > > On Thu, May 9, 2019 at 8:52 AM Andres Freund wrote: > > > > The changes necessary for tableam were already

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Ashwin Agrawal
On Fri, May 10, 2019 at 10:51 AM Andres Freund wrote: > > Hi, > > On 2019-05-10 10:43:44 -0700, Ashwin Agrawal wrote: > > On Thu, May 9, 2019 at 8:52 AM Andres Freund wrote: > > > The changes necessary for tableam were already huge. Changing naming > > > schemes for functions that are used all ov

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Andres Freund
Hi, On 2019-05-10 10:43:44 -0700, Ashwin Agrawal wrote: > On Thu, May 9, 2019 at 8:52 AM Andres Freund wrote: > > The changes necessary for tableam were already huge. Changing naming > > schemes for functions that are used all over the backend (e.g. ~80 calls > > to table_beginscan), and where th

Re: Inconsistency between table am callback and table function names

2019-05-10 Thread Ashwin Agrawal
On Thu, May 9, 2019 at 8:52 AM Andres Freund wrote: > The changes necessary for tableam were already huge. Changing naming > schemes for functions that are used all over the backend (e.g. ~80 calls > to table_beginscan), and where there's other wrapper functions that also > widely used (237 calls

Re: Inconsistency between table am callback and table function names

2019-05-09 Thread Andres Freund
Hi, On 2019-05-08 17:05:07 -0700, Ashwin Agrawal wrote: > On Wed, May 8, 2019 at 2:51 PM Andres Freund wrote: > > On 2019-05-08 00:32:22 -0700, Ashwin Agrawal wrote: > > > The general theme for table function names seem to be > > > "table_". For example table_scan_getnextslot() and its > > > corr

Re: Inconsistency between table am callback and table function names

2019-05-08 Thread Ashwin Agrawal
On Wed, May 8, 2019 at 2:51 PM Andres Freund wrote: > > Hi, > > On 2019-05-08 00:32:22 -0700, Ashwin Agrawal wrote: > > The general theme for table function names seem to be > > "table_". For example table_scan_getnextslot() and its > > corresponding callback scan_getnextslot(). Most of the table

Re: Inconsistency between table am callback and table function names

2019-05-08 Thread Andres Freund
Hi, On 2019-05-08 00:32:22 -0700, Ashwin Agrawal wrote: > The general theme for table function names seem to be > "table_". For example table_scan_getnextslot() and its > corresponding callback scan_getnextslot(). Most of the table functions and > callbacks follow mentioned convention except follo

Inconsistency between table am callback and table function names

2019-05-08 Thread Ashwin Agrawal
The general theme for table function names seem to be "table_". For example table_scan_getnextslot() and its corresponding callback scan_getnextslot(). Most of the table functions and callbacks follow mentioned convention except following ones table_beginscan table_endscan table_rescan table_f