Remove pltsql_call_handler from sys.all_objects (#2201) (#2206)

We have fixed duplicate OIDs for most objects, however, `pltsql_call_handler` still has a possibility of duplicate OIDs after MVU to PG14/15/16 because `GetNewObjectId_hook()` is not called when we restore the first C function. This commit removes pltsql_call_handler from sys.all_objects, hence sys.all_objects won't have duplicate OIDs after major version upgrades. (This is a cherry-pick from ef84a49 except .github/scripts/clone_engine_repo.conf) Task: BABEL-4662 Signed-off-by: Jungkook Lee <[email protected]>
babelfish-for-postgresql · Jan 3, 2024 · 4b46768 · 4b46768
1 parent d5911a4
commit 4b46768
Show file tree

Hide file tree

Showing 9 changed files with 445 additions and 181 deletions.
diff --git a/.github/scripts/clone_engine_repo.conf b/.github/scripts/clone_engine_repo.conf
@@ -1 +1 @@
-DEFAULT_BRANCH=BABEL_3_4_STABLE__PG_15_5
+DEFAULT_BRANCH=BABEL_4_0_STABLE__PG_16_1
diff --git a/contrib/babelfishpg_tsql/sql/sys_views.sql b/contrib/babelfishpg_tsql/sql/sys_views.sql
@@ -1504,6 +1504,7 @@ and nis.type = (case p.prokind
 where (s.nspname = 'sys' or (nis.name is not null and ext.nspname is not null))
 and has_schema_privilege(s.oid, 'USAGE')
 and has_function_privilege(p.oid, 'EXECUTE')
+and p.proname != 'pltsql_call_handler'
 
 union all
 -- details of user defined procedures

diff --git a/contrib/babelfishpg_tsql/sql/upgrades/babelfishpg_tsql--3.4.0--3.5.0.sql b/contrib/babelfishpg_tsql/sql/upgrades/babelfishpg_tsql--3.4.0--3.5.0.sql
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		DEFAULT_BRANCH=BABEL_3_4_STABLE__PG_15_5
		DEFAULT_BRANCH=BABEL_4_0_STABLE__PG_16_1