diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-03-28 12:52:55 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-03-28 19:02:23 -0300 |
| commit | ce96ce60ca2293f75f36c3661e4657a3c79ffd61 (patch) | |
| tree | 45247e9dd84a19acd6a2d024ddcc95007e12a577 /src/backend/replication/logical/reorderbuffer.c | |
| parent | a99f77021f0c8c1c221af4e36b64ca43abd04389 (diff) | |
| download | postgresql-ce96ce60ca2293f75f36c3661e4657a3c79ffd61.tar.gz postgresql-ce96ce60ca2293f75f36c3661e4657a3c79ffd61.zip | |
Remove direct uses of ItemPointer.{ip_blkid,ip_posid}
There are no functional changes here; this simply encapsulates knowledge
of the ItemPointerData struct so that a future patch can change things
without more breakage.
All direct users of ip_blkid and ip_posid are changed to use existing
macros ItemPointerGetBlockNumber and ItemPointerGetOffsetNumber
respectively. For callers where that's inappropriate (because they
Assert that the itempointer is is valid-looking), add
ItemPointerGetBlockNumberNoCheck and ItemPointerGetOffsetNumberNoCheck,
which lack the assertion but are otherwise identical.
Author: Pavan Deolasee
Discussion: https://postgr.es/m/CABOikdNnFon4cJiL=h1mZH3bgUeU+sWHuU4Yr8AB=j3A2p1GiA@mail.gmail.com
Diffstat (limited to 'src/backend/replication/logical/reorderbuffer.c')
| -rw-r--r-- | src/backend/replication/logical/reorderbuffer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c index b437799c5fd..12ebadca815 100644 --- a/src/backend/replication/logical/reorderbuffer.c +++ b/src/backend/replication/logical/reorderbuffer.c @@ -3013,8 +3013,8 @@ DisplayMapping(HTAB *tuplecid_data) ent->key.relnode.dbNode, ent->key.relnode.spcNode, ent->key.relnode.relNode, - BlockIdGetBlockNumber(&ent->key.tid.ip_blkid), - ent->key.tid.ip_posid, + ItemPointerGetBlockNumber(&ent->key.tid), + ItemPointerGetOffsetNumber(&ent->key.tid), ent->cmin, ent->cmax ); |