diff options
| author | Michael Paquier <michael@paquier.xyz> | 2022-02-17 09:52:02 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2022-02-17 09:52:02 +0900 |
| commit | d61a361d1aef1231db61162d99b635b89c73169d (patch) | |
| tree | 87cdc9beef527e6ff005748e5ef7029cfd302cce /src/backend/utils/adt/mcxtfuncs.c | |
| parent | 46208923446651a58c49115d17b4de6da4934c71 (diff) | |
| download | postgresql-d61a361d1aef1231db61162d99b635b89c73169d.tar.gz postgresql-d61a361d1aef1231db61162d99b635b89c73169d.zip | |
Remove all traces of tuplestore_donestoring() in the C code
This routine is a no-op since dd04e95 from 2003, with a macro kept
around for compatibility purposes. This has led to the same code
patterns being copy-pasted around for no effect, sometimes in confusing
ways like in pg_logical_slot_get_changes_guts() from logical.c where the
code was actually incorrect.
This issue has been discussed on two different threads recently, so
rather than living with this legacy, remove any uses of this routine in
the C code to simplify things. The compatibility macro is kept to avoid
breaking any out-of-core modules that depend on it.
Reported-by: Tatsuhito Kasahara, Justin Pryzby
Author: Tatsuhito Kasahara
Discussion: https://postgr.es/m/20211217200419.GQ17618@telsasoft.com
Discussion: https://postgr.es/m/CAP0=ZVJeeYfAeRfmzqAF2Lumdiv4S4FewyBnZd4DPTrsSQKJKw@mail.gmail.com
Diffstat (limited to 'src/backend/utils/adt/mcxtfuncs.c')
| -rw-r--r-- | src/backend/utils/adt/mcxtfuncs.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/backend/utils/adt/mcxtfuncs.c b/src/backend/utils/adt/mcxtfuncs.c index 28cb9d3ff10..c7c95adf97d 100644 --- a/src/backend/utils/adt/mcxtfuncs.c +++ b/src/backend/utils/adt/mcxtfuncs.c @@ -152,9 +152,6 @@ pg_get_backend_memory_contexts(PG_FUNCTION_ARGS) PutMemoryContextsStatsTupleStore(tupstore, tupdesc, TopMemoryContext, NULL, 0); - /* clean up and return the tuplestore */ - tuplestore_donestoring(tupstore); - return (Datum) 0; } |