On 12-11-14 08:17 PM, Andres Freund wrote:
For the regular satisfies routines this is needed in prepareation of logical
decoding. I changed the non-regular ones for consistency as well.
The naming between htup, tuple and similar is rather confused, I could not find
any consistent naming anywhere.
This is preparatory work for the logical decoding feature which needs to be
able to get to a valid relfilenode from when checking the visibility of a
tuple.
I have taken a look at this patch. The patch does what it says, it changes a bunch of
HeapTupleSatisfiesXXXX routines to take a HeapTuple structure instead of a HeapTupleHeader.
It also sets the HeapTuple.t_tableOid value before calling these routines.
The patch does not modify these routines to actually do anything useful with the additional data fields though it does add some assertions to make sure t_tableOid is set.
The patch compiles cleanly and the unit tests pass.
This patch does not seem to depend on any of the other patches in this set and applies cleanly against master. The patch doesn't actually add any functionality, unless someone sees a reason for complaining about this that I don't see, then I think it can be committed.
Steve
---contrib/pgrowlocks/pgrowlocks.c | 2 +-src/backend/access/heap/heapam.c | 13 ++++++----src/backend/access/heap/pruneheap.c | 16 ++++++++++--src/backend/catalog/index.c | 2 +-src/backend/commands/analyze.c | 3 ++-src/backend/commands/cluster.c | 2 +-src/backend/commands/vacuumlazy.c | 3 ++-src/backend/storage/lmgr/predicate.c | 2 +-src/backend/utils/time/tqual.c | 50 +++++++++++++++++++++++++++++-------src/include/utils/snapshot.h | 4 +--src/include/utils/tqual.h | 20 +++++++--------11 files changed, 83 insertions(+), 34 deletions(-)
