[GitHub] potiuk commented on issue #4538: [AIRFLOW-3629] remove abstract methods from base hook

[GitBox <gi...@apache.org>]

potiuk commented on issue #4538: [AIRFLOW-3629] remove abstract methods from base hook
URL: https://github.com/apache/airflow/pull/4538#issuecomment-454596458
 
 
   Actually - after seeing the comments I realise I know to little about why those methods are there in the first place. It's a bit risky to change it just to get rid of the warnings without full understanding. I do not know in the first place why those methods are there, and it seems that there might be some side effects, so I better abandon that.
   
   This is not really that important to get it fixed, I can simply (and my colleagues too) disable the warnings in IDE instead.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services