stikkireddy commented on code in PR #32221:
URL: https://github.com/apache/airflow/pull/32221#discussion_r1251247493
##########
airflow/providers/databricks/hooks/databricks.py:
##########
@@ -135,9 +137,27 @@ def __init__(
) -> None:
super().__init__(databricks_conn_id, timeout_seconds, retry_limit,
retry_delay, retry_args, caller)
+ def create(self, json: dict) -> int:
+ """
+ Utility function to call the ``api/2.1/jobs/create`` endpoint.
+
+ :param json: The data used in the body of the request to the
``create`` endpoint.
+ :return: the job_id as an int
Review Comment:
hey @pankajastro this has a return for the job id, are you saying to put a
docstring for when the return for a method is None? Such as the reset method?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
