Skip to content

feat: fix glue catalog quotes when setting quoting config to true#680

Open
chuachengling wants to merge 3 commits intoaws-samples:mainfrom
chuachengling:fix-glue-catalog-quotes
Open

feat: fix glue catalog quotes when setting quoting config to true#680
chuachengling wants to merge 3 commits intoaws-samples:mainfrom
chuachengling:fix-glue-catalog-quotes

Conversation

@chuachengling
Copy link
Copy Markdown

@chuachengling chuachengling commented Apr 17, 2026

resolves #675

Description

This PR resolves the issue where quoting configuration does not flow through for the glue statement during table materialization.

For example

Exception: 
  [INVALID_IDENTIFIER] The identifier glue-schema is invalid. Please, consider quoting it with back-quotes as `glue-schema`.(line 3, pos 29)
  
  == SQL ==
  /* {"app": "dbt", "dbt_version": "1.10.20", "profile_name": "glue_ecommerce", "target_name": "dev", "node_id": "model.glue_ecommerce.stg_customers"} */
  
      drop table if exists glue-schema.stg_customers
  -----------------------------^^^
  
15:27:09  
15:27:09    compiled code at target/compiled/glue_ecommerce/models/staging/stg_customers.sql
15:27:09  
15:27:09  Done. PASS=0 WARN=0 ERROR=1 SKIP=0 NO-OP=0 TOTAL=1

Checklist

  • I have signed the CLA
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt-glue next" section.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dbt glue does not quote schema in glue catalog

1 participant