diff options
author | Michael Paquier | 2018-11-06 05:11:21 +0000 |
---|---|---|
committer | Michael Paquier | 2018-11-06 05:11:21 +0000 |
commit | 8f045e242b2308473f5de532590454294e3d5683 (patch) | |
tree | 89837935be9ab9cbf3445d4874b47d10cd54ba84 | |
parent | 3c60d0fa231f7699af5dd83345919d0423305973 (diff) |
Switch pg_promote to be parallel-safe
pg_promote uses nothing relying on a global state, so it is fine to mark
it as parallel-safe, conclusion based on a detailed analysis from Robert
Haas. This also fixes an inconsistency where pg_proc.dat missed to mark
the function with its previous value for proparallel, update which does
not matter now as the default is used.
Based on a discussion between multiple folks: Laurenz Albe, Robert Haas,
Amit Kapila, Tom Lane and myself.
Discussion: https://postgr.es/m/20181029082530.GL14242@paquier.xyz
-rw-r--r-- | src/backend/catalog/system_views.sql | 2 | ||||
-rw-r--r-- | src/include/catalog/catversion.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/catalog/system_views.sql b/src/backend/catalog/system_views.sql index 53ddc593a8c..715995dd883 100644 --- a/src/backend/catalog/system_views.sql +++ b/src/backend/catalog/system_views.sql @@ -1030,7 +1030,7 @@ CREATE OR REPLACE FUNCTION pg_stop_backup ( CREATE OR REPLACE FUNCTION pg_promote(wait boolean DEFAULT true, wait_seconds integer DEFAULT 60) RETURNS boolean STRICT VOLATILE LANGUAGE INTERNAL AS 'pg_promote' - PARALLEL RESTRICTED; + PARALLEL SAFE; -- legacy definition for compatibility with 9.3 CREATE OR REPLACE FUNCTION diff --git a/src/include/catalog/catversion.h b/src/include/catalog/catversion.h index 14fc4ddda85..7d78cbe026a 100644 --- a/src/include/catalog/catversion.h +++ b/src/include/catalog/catversion.h @@ -53,6 +53,6 @@ */ /* yyyymmddN */ -#define CATALOG_VERSION_NO 201811051 +#define CATALOG_VERSION_NO 201811061 #endif |