[GitHub] [airflow] Seokyun-Ha commented on a diff in pull request #34643: DatabricksHook ClusterState

["Seokyun-Ha (via GitHub)" <gi...@apache.org>]

Seokyun-Ha commented on code in PR #34643:
URL: https://github.com/apache/airflow/pull/34643#discussion_r1339828039


##########
tests/providers/databricks/hooks/test_databricks.py:
##########
@@ -158,6 +162,11 @@ def repair_run_endpoint(host):
     """
     return f"https://{host}/api/2.1/jobs/runs/repair"
 
+def get_cluster_endpoint(host):
+    """
+    Utility function to generate the get run endpoint given the host.
+    """
+    return f"https://{host}/api/2.0/clusters/get"

Review Comment:
   I just follow the pattern of exiting codes in the file 🙏 



##########
tests/providers/databricks/hooks/test_databricks.py:
##########
@@ -158,6 +162,11 @@ def repair_run_endpoint(host):
     """
     return f"https://{host}/api/2.1/jobs/runs/repair"
 
+def get_cluster_endpoint(host):
+    """
+    Utility function to generate the get run endpoint given the host.
+    """
+    return f"https://{host}/api/2.0/clusters/get"

Review Comment:
   I just follow the pattern of exiting codes in the file 🙏 



##########
tests/providers/databricks/hooks/test_databricks.py:
##########
@@ -158,6 +162,11 @@ def repair_run_endpoint(host):
     """
     return f"https://{host}/api/2.1/jobs/runs/repair"
 
+def get_cluster_endpoint(host):
+    """
+    Utility function to generate the get run endpoint given the host.
+    """
+    return f"https://{host}/api/2.0/clusters/get"

Review Comment:
   I just followed the pattern of exiting codes in the file 🙏 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org