marcusianlevine commented on pull request #11509:
URL: https://github.com/apache/airflow/pull/11509#issuecomment-730441486


   Thanks for your help getting this in everyone, but it seems that the 
programmatic refactor didn't work as expected.
   
   I installed the latest backport provider release, but while the copy of the 
Airflow 2.0 `file_task_handler.py` file is copied into the proper location 
inside the installed backport provider, the provider's `log/es_task_handler.py` 
file was not properly refactored to import the `FileTaskHandler` from the 
correct module:
   
   ```
   $ cat 
/usr/local/lib/python3.7/site-packages/airflow/providers/elasticsearch/log/es_task_handler.py
 | grep file_task_handler
   from airflow.utils.log.file_task_handler import FileTaskHandler
   ```
   
   This block is supposed to select the appropriate files inside the provider 
to target for the refactor, but it doesn't seem to work: 
https://github.com/apache/airflow/pull/11509/files#diff-11dd8f34d8ae58d0c8cc576f298e4e83d40b512384d1e1707b6313f406b5f256R494


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to