pgsql: Release memory allocated by dependency_degree

From: Tomas Vondra <tomas(dot)vondra(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Release memory allocated by dependency_degree
Date: 2021-09-23 16:53:38
Message-ID: E1mTRyc-0005FZ-2q@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Release memory allocated by dependency_degree

Calculating degree of a functional dependency may allocate a lot of
memory - we have released mot of the explicitly allocated memory, but
e.g. detoasted varlena values were left behind. That may be an issue,
because we consider a lot of dependencies (all combinations), and the
detoasting may happen for each one again.

Fixed by calling dependency_degree() in a dedicated context, and
resetting it after each call. We only need the calculated dependency
degree, so we don't need to copy anything.

Backpatch to PostgreSQL 10, where extended statistics were introduced.

Backpatch-through: 10
Discussion: https://www.postgresql.org/message-id/20210915200928.GP831%40telsasoft.com

Branch
------
REL_11_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/4487a7def3ae6efa79c545c3edc03ea97f446e36

Modified Files
--------------
src/backend/statistics/dependencies.c | 22 +++++++++++++++++-----
1 file changed, 17 insertions(+), 5 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tomas Vondra 2021-09-23 17:01:04 pgsql: Release memory allocated by dependency_degree
Previous Message Tomas Vondra 2021-09-23 16:47:32 pgsql: Free memory after building each statistics object