diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-05-15 14:13:09 -0400 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-05-15 14:13:09 -0400 |
| commit | cafde58b337e007cb6a719f5ab4dd6459d932a39 (patch) | |
| tree | 4c91ed4a899e03f7b9f0f5cadf01f2eb8d9cb950 /src/backend/tcop/postgres.c | |
| parent | 30d8bad494ad1f604295033e4f4de4b8f258fe74 (diff) | |
| download | postgresql-cafde58b337e007cb6a719f5ab4dd6459d932a39.tar.gz postgresql-cafde58b337e007cb6a719f5ab4dd6459d932a39.zip | |
Allow compute_query_id to be set to 'auto' and make it default
Allowing only on/off meant that all either all existing configuration
guides would become obsolete if we disabled it by default, or that we
would have to accept a performance loss in the default config if we
enabled it by default. By allowing 'auto' as a middle ground, the
performance cost is only paid by those who enable pg_stat_statements and
similar modules.
I only edited the release notes to comment-out a paragraph that is now
factually wrong; further edits are probably needed to describe the
related change in more detail.
Author: Julien Rouhaud <rjuju123@gmail.com>
Reviewed-by: Justin Pryzby <pryzby@telsasoft.com>
Discussion: https://postgr.es/m/20210513002623.eugftm4nk2lvvks3@nol
Diffstat (limited to 'src/backend/tcop/postgres.c')
| -rw-r--r-- | src/backend/tcop/postgres.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index dd2ade7bb65..8cea10c9019 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -704,7 +704,7 @@ pg_analyze_and_rewrite_params(RawStmt *parsetree, query = transformTopLevelStmt(pstate, parsetree); - if (compute_query_id) + if (IsQueryIdEnabled()) jstate = JumbleQuery(query, query_string); if (post_parse_analyze_hook) |