Skip to content
This repository was archived by the owner on Sep 2, 2025. It is now read-only.
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
initialize sqlparse Lexer prior to using sqlparse
  • Loading branch information
mikealfare committed Feb 6, 2024
commit 41c44ca6168b3ce493edca1b48835ab4a25d4219
12 changes: 12 additions & 0 deletions dbt/adapters/redshift/connections.py
Original file line number Diff line number Diff line change
Expand Up @@ -354,6 +354,7 @@ def add_query(self, sql, auto_begin=True, bindings=None, abridge_sql_log=False):
connection = None
cursor = None

self._initialize_sqlparse_lexer()
queries = sqlparse.split(sql)

for query in queries:
Expand Down Expand Up @@ -385,3 +386,14 @@ def get_credentials(cls, credentials):
@classmethod
def data_type_code_to_name(cls, type_code: Union[int, str]) -> str:
return get_datatype_name(type_code)

@staticmethod
def _initialize_sqlparse_lexer():
"""
Resolves: https://github.com/dbt-labs/dbt-redshift/issues/710
Implementation of this fix: https://github.com/dbt-labs/dbt-core/pull/8215
"""
from sqlparse.lexer import Lexer # type: ignore

if hasattr(Lexer, "get_default_instance"):
Lexer.get_default_instance()