From 430bc25db1cf236da3ea2df14b0844d3d156fc59 Mon Sep 17 00:00:00 2001 From: Jensen Yap Date: Sat, 16 Aug 2025 02:26:46 +0900 Subject: [PATCH] Refactor external access integration and clean up SQL macros (#129) - Removed the unused external access SQL file to streamline the project structure. - Updated the S3 express external access integration macro by eliminating unnecessary schema usage. - Adjusted dependencies in the utils SQL file to reflect the removal of the external access integration. --- macros/livequery/external_access_integrations.sql | 4 ---- models/deploy/core/_external_access.sql | 8 -------- models/deploy/core/utils.sql | 1 - 3 files changed, 13 deletions(-) delete mode 100644 models/deploy/core/_external_access.sql diff --git a/macros/livequery/external_access_integrations.sql b/macros/livequery/external_access_integrations.sql index a561591..f6dd320 100644 --- a/macros/livequery/external_access_integrations.sql +++ b/macros/livequery/external_access_integrations.sql @@ -1,8 +1,4 @@ {% macro create_s3_express_external_access_integration() %} - {% set use_schema_sql %} - USE SCHEMA live - {% endset %} - {% set network_rule_sql %} CREATE NETWORK RULE IF NOT EXISTS s3_express_network_rule MODE = EGRESS diff --git a/models/deploy/core/_external_access.sql b/models/deploy/core/_external_access.sql deleted file mode 100644 index 2740d86..0000000 --- a/models/deploy/core/_external_access.sql +++ /dev/null @@ -1,8 +0,0 @@ -{% if is_udf_api_v2_compatible() %} -{{ create_s3_express_external_access_integration() }} -{% endif %} - --- this is to pass the model render as dbt dependency in other models --- livequery will need s3 express access to read from the s3 bucket - -SELECT 1 diff --git a/models/deploy/core/utils.sql b/models/deploy/core/utils.sql index 2f204c0..fbabfe1 100644 --- a/models/deploy/core/utils.sql +++ b/models/deploy/core/utils.sql @@ -1,4 +1,3 @@ -- depends_on: {{ ref('_utils') }} - -- depends_on: {{ ref('_external_access')}} {% set config = config_core_utils %} {{ ephemeral_deploy_core(config) }}