ephraimbuddy commented on pull request #9879:
URL: https://github.com/apache/airflow/pull/9879#issuecomment-717848935


   > I thoroughly rewritten the implementation to be compatible with the SSH 
Operator and also to make the logic much simpler. I think the code is perfect 
now. I managed to solve all the problems that I encountered and the system 
tests are passing correctly. Especially SSH tunneling was problematic as SSH 
tunnel runs in an independent subprocess and I didn't see any error messages so 
I didn't know you should get authorization separately but now everything works.
   > 
   > I still have a documentation update + some unit tests.
   > 
   > @ephraimbuddy What do you think about the current implementation? Is it 
clearer?
   
   Wow! This is much better. I gave up on this after so many attempts. Thanks a 
bunch!.


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