You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/04/22 09:49:35 UTC

[GitHub] [airflow] zhongjiajie commented on a change in pull request #8475: [AIRFLOW-8474]: Adding possibility to get job_id from Databricks run

zhongjiajie commented on a change in pull request #8475:
URL: https://github.com/apache/airflow/pull/8475#discussion_r412836134



##########
File path: airflow/providers/databricks/hooks/databricks.py
##########
@@ -236,6 +236,18 @@ def get_run_page_url(self, run_id: str) -> str:
         response = self._do_api_call(GET_RUN_ENDPOINT, json)
         return response['run_page_url']
 
+    def get_job_id_from_run(self, run_id: str) -> str:

Review comment:
       ```suggestion
       def get_job_id(self, run_id: str) -> str:
   ```
   Maybe better.




----------------------------------------------------------------
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.

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