ashb commented on a change in pull request #21301:
URL: https://github.com/apache/airflow/pull/21301#discussion_r809225721
##########
File path: airflow/executors/base_executor.py
##########
@@ -310,3 +313,14 @@ def debug_dump(self):
len(self.event_buffer),
"\n\t".join(map(repr, self.event_buffer.items())),
)
+
+ def send_callback_to_execute(self, request: CallbackRequest) -> None:
+ """Sends callback for execution.
+
+ Provides a default implementation which sends the callback to
DagProcessor.
Review comment:
_This_ class doesn't have that default, and this implementation is
likely good enough for most executors, so we should re-word this docstring.
--
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]
