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


   > The deprecation warnings should be added to "known" list and tests are 
failing unfortunately :)
   
   Hey @potiuk , can you point me the way here...
   
   Providers manager is warning here:
   
   ```
                           if already_registered.hook_class_name != 
hook_class_name:
                               log.warning(
                                   "The hook connection type '%s' is registered 
twice in the"
                                   " package '%s' with different class names: 
'%s' and '%s'. "
                                   " Please fix it!",
                                   hook_info.connection_type,
                                   package_name,
                                   already_registered.hook_class_name,
                                   hook_class_name,
                               )
   ```
   
   This causes a test to fail.  I did not change anything about the class names 
or connection types.  They're now in different modules though.
   
   Do you know what to do here?  Perhaps we should not have let `conn_type` be 
`redshift` for the RedshiftSqlHook when it was added.
   
   


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