summaryrefslogtreecommitdiff
path: root/contrib/ltree_plpython
diff options
context:
space:
mode:
authorTom Lane2016-01-12 01:06:36 +0000
committerTom Lane2016-01-12 01:06:36 +0000
commitfb6fcbd33fbbd6d31fa2b39938e60ecb48dc4de4 (patch)
tree7e7e1557c38a52e6f236f649cc0296170253d339 /contrib/ltree_plpython
parent866566a690bb9916dcd294807e65a6e173396530 (diff)
Use LOAD not actual code execution to pull in plpython library.
Commit 866566a690bb9916 is insufficient to prevent dump/reload failures when using transform modules in a database with both plpython2 and plpython3 installed. The reason is that the transform extension scripts use DO blocks as a mechanism to pull in the libpython library before creating the transform function. It's necessary to preload the library because the dynamic loader won't do it for us on every platform, leading to "unresolved symbol" failures when the transform library is loaded. But it's *not* necessary to execute Python code, and doing so will provoke a multiple-Pythons-are-loaded error even after the preceding commit. To fix, use LOAD instead of a DO block. That requires superuser privilege, but creation of a C function does anyway. It also embeds knowledge of the underlying library name for each PL language; but that's wired into the initdb-time contents of pg_pltemplate too, so that doesn't seem like a large problem either. Note that CREATE TRANSFORM as such doesn't call the language module at all. Per a report from Paul Jones. Back-patch to 9.5 where transform modules were introduced.
Diffstat (limited to 'contrib/ltree_plpython')
-rw-r--r--contrib/ltree_plpython/ltree_plpython2u--1.0.sql2
-rw-r--r--contrib/ltree_plpython/ltree_plpython3u--1.0.sql2
-rw-r--r--contrib/ltree_plpython/ltree_plpythonu--1.0.sql2
3 files changed, 3 insertions, 3 deletions
diff --git a/contrib/ltree_plpython/ltree_plpython2u--1.0.sql b/contrib/ltree_plpython/ltree_plpython2u--1.0.sql
index 29a12d45a28..f040bd3f561 100644
--- a/contrib/ltree_plpython/ltree_plpython2u--1.0.sql
+++ b/contrib/ltree_plpython/ltree_plpython2u--1.0.sql
@@ -1,5 +1,5 @@
-- make sure the prerequisite libraries are loaded
-DO '1' LANGUAGE plpython2u;
+LOAD 'plpython2';
SELECT NULL::ltree;
diff --git a/contrib/ltree_plpython/ltree_plpython3u--1.0.sql b/contrib/ltree_plpython/ltree_plpython3u--1.0.sql
index 1300a78c66f..7afe51f1483 100644
--- a/contrib/ltree_plpython/ltree_plpython3u--1.0.sql
+++ b/contrib/ltree_plpython/ltree_plpython3u--1.0.sql
@@ -1,5 +1,5 @@
-- make sure the prerequisite libraries are loaded
-DO '1' LANGUAGE plpython3u;
+LOAD 'plpython3';
SELECT NULL::ltree;
diff --git a/contrib/ltree_plpython/ltree_plpythonu--1.0.sql b/contrib/ltree_plpython/ltree_plpythonu--1.0.sql
index 1d1af28f4eb..50f35dd16b0 100644
--- a/contrib/ltree_plpython/ltree_plpythonu--1.0.sql
+++ b/contrib/ltree_plpython/ltree_plpythonu--1.0.sql
@@ -1,5 +1,5 @@
-- make sure the prerequisite libraries are loaded
-DO '1' LANGUAGE plpythonu;
+LOAD 'plpython2'; -- change to plpython3 if that ever becomes the default
SELECT NULL::ltree;