Merge branch 'main' into dependabot/pip/black-approx-eq-23.10

.changes/unreleased/Dependencies-20230912-120620.yaml

@@ -0,0 +1,6 @@
+kind: Dependencies
+body: Update redshift-connector requirement from ~=2.0.913 to ~=2.0.914
+time: 2023-09-12T12:06:20.401643-07:00
+custom:
+  Author: soksamnanglim
+  PR: "601"

.changes/unreleased/Fixes-20231025-203732.yaml

@@ -0,0 +1,6 @@
+kind: Fixes
+body: Fix parsing of database results for materialized view auto refresh
+time: 2023-10-25T20:37:32.191259-04:00
+custom:
+  Author: mikealfare
+  Issue: "643"

dbt/adapters/redshift/relation_configs/materialized_view.py

@@ -182,10 +182,14 @@ def parse_relation_results(cls, relation_results: RelationResults) -> dict:
             "mv_name": materialized_view.get("table"),
             "schema_name": materialized_view.get("schema"),
             "database_name": materialized_view.get("database"),
-            "autorefresh": materialized_view.get("autorefresh"),
             "query": cls._parse_query(query.get("definition")),
         }
 
+        autorefresh_value = materialized_view.get("autorefresh")
+        if autorefresh_value is not None:
+            bool_filter = {"t": True, "f": False}
+            config_dict["autorefresh"] = bool_filter.get(autorefresh_value, autorefresh_value)
+
         # the default for materialized views differs from the default for diststyle in general
         # only set it if we got a value
         if materialized_view.get("diststyle"):

setup.py

@@ -87,7 +87,7 @@ def _core_version(plugin_version: str = _plugin_version()) -> str:
         "boto3~=1.26.157",
         # dbt-redshift depends deeply on this package. it does not follow SemVer, therefore there have been breaking changes in previous patch releases
         # Pin to the patch or minor version, and bump in each new minor version of dbt-redshift.
-        "redshift-connector==2.0.913",
+        "redshift-connector==2.0.915",
         # installed via dbt-core but referenced directly; don't pin to avoid version conflicts with dbt-core
         "agate",
     ],

tests/functional/adapter/materialized_view_tests/test_materialized_views.py

@@ -163,19 +163,6 @@ def test_change_is_applied_via_alter_str_true(self, project, my_materialized_vie
         assert_message_in_logs(f"Applying ALTER to: {my_materialized_view}", logs)
         assert_message_in_logs(f"Applying REPLACE to: {my_materialized_view}", logs, False)
 
-    def test_change_is_applied_via_alter_str_false(self, project, my_materialized_view):
-        self.check_start_state(project, my_materialized_view)
-
-        self.change_config_via_alter_str_false(project, my_materialized_view)
-        _, logs = run_dbt_and_capture_with_retries_redshift_mv(
-            ["--debug", "run", "--models", my_materialized_view.name]
-        )
-
-        self.check_state_alter_change_is_applied_str_false(project, my_materialized_view)
-
-        assert_message_in_logs(f"Applying ALTER to: {my_materialized_view}", logs)
-        assert_message_in_logs(f"Applying REPLACE to: {my_materialized_view}", logs, False)
-
     def test_change_is_applied_via_replace(self, project, my_materialized_view):
         self.check_start_state(project, my_materialized_view)