diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2024-08-14 08:02:32 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2024-08-14 08:07:52 +0200 |
| commit | c8e2d422fd556292ab751392bf76f713fe9e9fc1 (patch) | |
| tree | 509e5ca50e69cd08ead2e2c85bd32c191ce09145 /src/backend/utils/adt/varlena.c | |
| parent | bf3401fe813ad3b6b34e5e9daf8fbd03540a2294 (diff) | |
| download | postgresql-c8e2d422fd556292ab751392bf76f713fe9e9fc1.tar.gz postgresql-c8e2d422fd556292ab751392bf76f713fe9e9fc1.zip | |
Remove TRACE_SORT macro
The TRACE_SORT macro guarded the availability of the trace_sort GUC
setting. But it has been enabled by default ever since it was
introduced in PostgreSQL 8.1, and there have been no reports that
someone wanted to disable it. So just remove the macro to simplify
things. (For the avoidance of doubt: The trace_sort GUC is still
there. This only removes the rarely-used macro guarding it.)
Reviewed-by: Heikki Linnakangas <hlinnaka@iki.fi>
Discussion: https://www.postgresql.org/message-id/flat/be5f7162-7c1d-44e3-9a78-74dcaa6529f2%40eisentraut.org
Diffstat (limited to 'src/backend/utils/adt/varlena.c')
| -rw-r--r-- | src/backend/utils/adt/varlena.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/backend/utils/adt/varlena.c b/src/backend/utils/adt/varlena.c index 52ab8c43c66..4f9a676c939 100644 --- a/src/backend/utils/adt/varlena.c +++ b/src/backend/utils/adt/varlena.c @@ -2455,7 +2455,6 @@ varstr_abbrev_abort(int memtupcount, SortSupport ssup) * time there are differences within full key strings not captured in * abbreviations. */ -#ifdef TRACE_SORT if (trace_sort) { double norm_abbrev_card = abbrev_distinct / (double) memtupcount; @@ -2465,7 +2464,6 @@ varstr_abbrev_abort(int memtupcount, SortSupport ssup) memtupcount, abbrev_distinct, key_distinct, norm_abbrev_card, sss->prop_card); } -#endif /* * If the number of distinct abbreviated keys approximately matches the @@ -2527,12 +2525,10 @@ varstr_abbrev_abort(int memtupcount, SortSupport ssup) * of moderately high to high abbreviated cardinality. There is little to * lose but much to gain, which our strategy reflects. */ -#ifdef TRACE_SORT if (trace_sort) elog(LOG, "varstr_abbrev: aborted abbreviation at %d " "(abbrev_distinct: %f, key_distinct: %f, prop_card: %f)", memtupcount, abbrev_distinct, key_distinct, sss->prop_card); -#endif return true; } |