dstandish commented on pull request #6850:
URL: https://github.com/apache/airflow/pull/6850#issuecomment-768548513


   @ldacey let me add...
   
   
   just going throught the process of getting something merged is meaningful 
and valuable work in itself and this would probably be a nice addition... one 
area where you could add creative input is thinking through wherther and how to 
make it so mssqloperator can use turbodbc optionally.... right now mssql 
operator will use odbc hook if conn type is `odbc` and original mssql hook 
otherwise.... you could maybe make it so if conn type is `turbodbc` then it 
uses turbodbc connection...
   
   alternatively you could make a turbodbc hook that is a thin subclass of odbc 
hook... these decisions are unanswered and it's why i never made this pr.... 
you can contribute meaningfully by proposing a solution and shepherding through


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