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 2021/11/21 07:28:38 UTC

[GitHub] [airflow] uranusjr commented on a change in pull request #19723: Databricks: add more methods to represent run state information

uranusjr commented on a change in pull request #19723:
URL: https://github.com/apache/airflow/pull/19723#discussion_r753759296



##########
File path: airflow/providers/databricks/hooks/databricks.py
##########
@@ -400,7 +402,13 @@ def get_job_id(self, run_id: str) -> str:
 
     def get_run_state(self, run_id: str) -> RunState:
         """
-        Retrieves run state of the run.
+        Retrieves run state of the run. Please note that any Airflow tasks that call the `get_run_state`

Review comment:
       ```suggestion
           """Retrieves run state of the run.
   
           Please note that any Airflow tasks that call the `get_run_state`
   ```
   
   https://www.python.org/dev/peps/pep-0257/#multi-line-docstrings
   
   > Multi-line docstrings consist of a summary line just like a one-line docstring, followed by a blank line, followed by a more elaborate description.




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