Skip to content

Commit e687d87

Browse files
authored
refactor(redshift): read grants behavior flag directly (#2095)
1 parent 3915c53 commit e687d87

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

dbt-redshift/src/dbt/adapters/redshift/impl.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -208,11 +208,6 @@ def drop_without_cascade(self) -> bool:
208208
"""
209209
return bool(self.config.credentials.drop_without_cascade)
210210

211-
@available
212-
def use_grants_extended(self) -> bool:
213-
"""Whether to use extended grants support for groups and roles."""
214-
return self.behavior.redshift_grants_extended.no_warn
215-
216211
@available
217212
def verify_database(self, database):
218213
if database.startswith('"'):
@@ -362,7 +357,7 @@ def standardize_grants_dict(self, grants_table: "agate.Table") -> dict:
362357
grants_dict: Dict[str, List[str]] = {}
363358
current_user = self.config.credentials.user.lower()
364359

365-
if not self.use_grants_extended():
360+
if not self.behavior.redshift_grants_extended.no_warn:
366361
if not self.use_show_apis():
367362
# pg_user query returns a 'grantee' column — delegate to base.
368363
return super().standardize_grants_dict(grants_table)

dbt-redshift/src/dbt/include/redshift/macros/metadata/helpers.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
{% endmacro %}
44

55
{% macro redshift__use_grants_extended() %}
6-
{{ return(adapter.use_grants_extended()) }}
6+
{{ return(adapter.behavior.redshift_grants_extended.no_warn) }}
77
{% endmacro %}
88

99
{% macro redshift__drop_without_cascade() %}

0 commit comments

Comments
 (0)