On Thu, May 9, 2019 at 8:52 AM Andres Freund <and...@anarazel.de> 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 to systable_beginscan) which didn't have to be > touched, at the same time would have made it even harder to review.
If there are no objections to renaming now, as separate independent patch, I am happy to do the same and send it across. Will rename to make it consistent as mentioned at start of the thread leaving table_relation_xxx() ones as is today.
