Re: What is an item pointer, anyway? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: What is an item pointer, anyway?
Date
Msg-id 9915.1557776279@sss.pgh.pa.us
Whole thread Raw
In response to Re: What is an item pointer, anyway?  (Peter Geoghegan <pg@bowt.ie>)
Responses Re: What is an item pointer, anyway?  (Peter Geoghegan <pg@bowt.ie>)
List pgsql-hackers
Peter Geoghegan <pg@bowt.ie> writes:
> On Sun, May 5, 2019 at 1:14 PM Peter Geoghegan <pg@bowt.ie> wrote:
>> Attached draft patch adjusts code comments and error messages where a
>> line pointer is referred to as an item pointer. It turns out that this
>> practice isn't all that prevalent. Here are some specific concerns
>> that I had to think about when writing the patch, though:

> Ping? Any objections to pushing ahead with this?

Patch looks fine to me.  One minor quibble: in pruneheap.c you have

 /*
- * Prune specified item pointer or a HOT chain originating at that item.
+ * Prune specified line pointer or a HOT chain originating at that item.
  *
  * If the item is an index-referenced tuple (i.e. not a heap-only tuple),

Should "that item" also be re-worded, for consistency?

            regards, tom lane



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: PG12, PGXS and linking pgfeutils
Next
From: Robert Haas
Date:
Subject: Re: Inconsistency between table am callback and table function names