diff options
| author | Tom Lane | 2017-02-01 22:18:36 +0000 |
|---|---|---|
| committer | Tom Lane | 2017-02-01 22:18:36 +0000 |
| commit | aedd554f84bb3cedb32e6e2a954a70537a4da6b9 (patch) | |
| tree | c958b5ed639cf0359136109198f9d2e5189ba5b7 /src/include | |
| parent | ab02896510e26e46b830c87eef2c03dd3c52c09e (diff) | |
Fix CatalogTupleInsert/Update abstraction for case of shared indstate.
Add CatalogTupleInsertWithInfo and CatalogTupleUpdateWithInfo to let
callers use the CatalogTupleXXX abstraction layer even in cases where
we want to share the results of CatalogOpenIndexes across multiple
inserts/updates for efficiency. This finishes the job begun in commit
2f5c9d9c9, by allowing some remaining simple_heap_insert/update
calls to be replaced. The abstraction layer is now complete enough
that we don't have to export CatalogIndexInsert at all anymore.
Also, this fixes several places in which 2f5c9d9c9 introduced performance
regressions by using retail CatalogTupleInsert or CatalogTupleUpdate even
though the previous coding had been able to amortize CatalogOpenIndexes
work across multiple tuples.
A possible future improvement is to arrange for the indexing.c functions
to cache the CatalogIndexState somewhere, maybe in the relcache, in which
case we could get rid of CatalogTupleInsertWithInfo and
CatalogTupleUpdateWithInfo again. But that's a task for another day.
Discussion: https://postgr.es/m/27502.1485981379@sss.pgh.pa.us
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/catalog/indexing.h | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/include/catalog/indexing.h b/src/include/catalog/indexing.h index 9d02666ed1b..6bce7328a28 100644 --- a/src/include/catalog/indexing.h +++ b/src/include/catalog/indexing.h @@ -30,11 +30,14 @@ typedef struct ResultRelInfo *CatalogIndexState; */ extern CatalogIndexState CatalogOpenIndexes(Relation heapRel); extern void CatalogCloseIndexes(CatalogIndexState indstate); -extern void CatalogIndexInsert(CatalogIndexState indstate, - HeapTuple heapTuple); -extern Oid CatalogTupleInsert(Relation heapRel, HeapTuple tup); +extern Oid CatalogTupleInsert(Relation heapRel, HeapTuple tup); +extern Oid CatalogTupleInsertWithInfo(Relation heapRel, HeapTuple tup, + CatalogIndexState indstate); extern void CatalogTupleUpdate(Relation heapRel, ItemPointer otid, HeapTuple tup); +extern void CatalogTupleUpdateWithInfo(Relation heapRel, + ItemPointer otid, HeapTuple tup, + CatalogIndexState indstate); extern void CatalogTupleDelete(Relation heapRel, ItemPointer tid); |