Adaverse commented on code in PR #48491:
URL: https://github.com/apache/airflow/pull/48491#discussion_r2025399617
##########
providers/microsoft/azure/src/airflow/providers/microsoft/azure/log/wasb_task_handler.py:
##########
@@ -210,3 +171,80 @@ def wasb_write(self, log: str, remote_log_location: str,
append: bool = True) ->
self.log.exception("Could not write logs to %s",
remote_log_location)
return False
return True
+
+
+class WasbTaskHandler(FileTaskHandler, LoggingMixin):
Review Comment:
All handlers seem to have the same code with few overrides, leading to
repetition. Can we extend these from `BaseRemoteTaskHandler` to remove this
code repetition?
--
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]
