[GitHub] [airflow] Lee-W commented on a diff in pull request #32785: adds support for querying Redshift Serverless clusters

2023-07-24 Thread via GitHub
Lee-W commented on code in PR #32785: URL: https://github.com/apache/airflow/pull/32785#discussion_r1272382745 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -60,13 +60,17 @@ def execute_query( with_event: bool = False, wait_for_completion: bool

[GitHub] [airflow] Lee-W commented on a diff in pull request #32785: adds support for querying Redshift Serverless clusters

2023-07-24 Thread via GitHub
Lee-W commented on code in PR #32785: URL: https://github.com/apache/airflow/pull/32785#discussion_r1272312032 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -95,6 +100,12 @@ def execute_query( statement_id = resp["Id"] +if kwargs["ClusterIden

[GitHub] [airflow] Lee-W commented on a diff in pull request #32785: adds support for querying Redshift Serverless clusters

2023-07-24 Thread via GitHub
Lee-W commented on code in PR #32785: URL: https://github.com/apache/airflow/pull/32785#discussion_r1272312032 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -95,6 +100,12 @@ def execute_query( statement_id = resp["Id"] +if kwargs["ClusterIden

[GitHub] [airflow] Lee-W commented on a diff in pull request #32785: adds support for querying Redshift Serverless clusters

2023-07-24 Thread via GitHub
Lee-W commented on code in PR #32785: URL: https://github.com/apache/airflow/pull/32785#discussion_r1271951686 ## airflow/providers/amazon/aws/hooks/redshift_data.py: ## @@ -53,6 +53,7 @@ def execute_query( database: str, sql: str | list[str], cluster_