jason810496 opened a new pull request, #54813:
URL: https://github.com/apache/airflow/pull/54813

   
   related: [Fix CloudwatchTaskHandler display error 
#54054](https://github.com/apache/airflow/pull/54054)
   
   ## Why
   
   While fixing comments in #54054, I realized that we should support `stream` 
method in `RemoteIO` as 
https://github.com/apache/airflow/pull/54054#pullrequestreview-3083573995 point 
out. Starting with Airflow 3, 
[FileTaskHandler._read_remote_logs](https://github.com/apache/airflow/blob/f7f32544726672ada2766fb76efcabfba4191cb8/airflow-core/src/airflow/utils/log/file_task_handler.py#L908-L909)
 will utilize `RemoteIO` behind the scenes, rather than relying on 
`<Provider>TaskHandler._read_remote_logs`.
   
   ## What
   
   Add `stream` method to `RemoteIO` Protocol and add corresponding unit tests.
   


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

Reply via email to