Re: Table AM callbacks referring to heap in declarations (+typos)
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Michael Paquier <michael@paquier.xyz>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-06-01T19:43:11Z
Lists: pgsql-hackers
On 2019-06-01 15:37:43 -0400, Michael Paquier wrote: > On Sat, Jun 01, 2019 at 12:22:10PM -0700, Andres Freund wrote: > > On 2019-06-01 15:09:46 -0400, Michael Paquier wrote: > >> While going through the table AM callbacks, I have bumped into a > >> couple of references to heap. I think that we should make that more > >> generic by using the term "table" as done when opening relations and > >> such. Attached is a cleanup patch. > > > > I'm unbothered by this, but I'm also not opposed to changing this. It's > > largely just keeping the previous code / comment. > > > >> While on it, I found a set of typos which looked like a copy-pasto > >> which got spread => "index_nfo". > > > > Yea, we should fix this. > > Thanks. Do you mind if I fix both then? I don't mind at all (although it's imo not a fix for the s/heap/table)! - Andres
Commits
-
Fix some typos and inconsistencies in tableam.h
- 041a2642e51e 12.0 landed