Fix TAM handling in update script

The TAM SQL code was not written with update and downgrade scripts
in mind prevents further releases past 2.18.0 due to not splitting
up the parts that need to be part of every update script and those
that can only run once during initial installation.

cmake/ScriptFiles.cmake

@@ -15,19 +15,21 @@ set(PRE_INSTALL_SOURCE_FILES
     pre_install/types.functions.sql
     pre_install/types.post.sql # Must be before tables.sql
     pre_install/tables.sql
+    pre_install/tam_functions.sql
+    pre_install/tam.sql
     pre_install/cache.sql
     pre_install/insert_data.sql)
 
 # Source files that define functions and need to be rerun in update
 set(PRE_INSTALL_FUNCTION_FILES
     pre_install/types.functions.sql
+    pre_install/tam_functions.sql
     )
 
 # The rest of the source files defining mostly functions
 set(SOURCE_FILES
     hypertable.sql
     chunk.sql
-    hypercore.sql
     ddl_internal.sql
     util_time.sql
     util_internal_table_ddl.sql
@@ -56,7 +58,8 @@ set(SOURCE_FILES
     cagg_migrate.sql
     job_stat_history_log_retention.sql
     osm_api.sql
-    compression_defaults.sql)
+    compression_defaults.sql
+    )
 
 if(ENABLE_DEBUG_UTILS AND CMAKE_BUILD_TYPE MATCHES Debug)
   list(APPEND SOURCE_FILES debug_build_utils.sql)

sql/hypercore.sql → sql/pre_install/tam.sql

@@ -2,16 +2,10 @@
 -- Please see the included NOTICE for copyright information and
 -- LICENSE-APACHE for a copy of the license.
 
-CREATE FUNCTION ts_hypercore_handler(internal) RETURNS table_am_handler
-AS '@MODULE_PATHNAME@', 'ts_hypercore_handler' LANGUAGE C;
-
-CREATE ACCESS METHOD hypercore TYPE TABLE HANDLER ts_hypercore_handler;
+CREATE ACCESS METHOD hypercore TYPE TABLE HANDLER @[email protected]_hypercore_handler;
 COMMENT ON ACCESS METHOD hypercore IS 'Storage engine using hybrid row/columnar compression';
 
-CREATE FUNCTION ts_hypercore_proxy_handler(internal) RETURNS index_am_handler
-AS '@MODULE_PATHNAME@', 'ts_hypercore_proxy_handler' LANGUAGE C;
-
-CREATE ACCESS METHOD hypercore_proxy TYPE INDEX HANDLER ts_hypercore_proxy_handler;
+CREATE ACCESS METHOD hypercore_proxy TYPE INDEX HANDLER @[email protected]_hypercore_proxy_handler;
 COMMENT ON ACCESS METHOD hypercore_proxy IS 'Hypercore proxy index access method';
 
 -- An index AM needs at least one operator class for the column type

sql/pre_install/tam_functions.sql

@@ -0,0 +1,10 @@
+-- This file and its contents are licensed under the Apache License 2.0.
+-- Please see the included NOTICE for copyright information and
+-- LICENSE-APACHE for a copy of the license.
+
+CREATE OR REPLACE FUNCTION @[email protected]_hypercore_handler(internal) RETURNS table_am_handler
+AS '@MODULE_PATHNAME@', 'ts_hypercore_handler' LANGUAGE C;
+
+CREATE OR REPLACE FUNCTION @[email protected]_hypercore_proxy_handler(internal) RETURNS index_am_handler
+AS '@MODULE_PATHNAME@', 'ts_hypercore_proxy_handler' LANGUAGE C;
+

sql/updates/reverse-dev.sql

@@ -1,8 +1,8 @@
 -- Hypercore AM
 DROP ACCESS METHOD IF EXISTS hypercore_proxy;
-DROP FUNCTION IF EXISTS ts_hypercore_proxy_handler;
+DROP FUNCTION IF EXISTS @extschema@.ts_hypercore_proxy_handler;
 DROP ACCESS METHOD IF EXISTS hypercore;
-DROP FUNCTION IF EXISTS ts_hypercore_handler;
+DROP FUNCTION IF EXISTS @extschema@.ts_hypercore_handler;
 DROP FUNCTION IF EXISTS _timescaledb_debug.is_compressed_tid;
 
 DROP FUNCTION IF EXISTS @[email protected]_chunk(uncompressed_chunk REGCLASS,	if_not_compressed BOOLEAN, recompress BOOLEAN, hypercore_use_access_method BOOL);