BasPH commented on a change in pull request #5406: [AIRFLOW-4681] Fix pylint
problems for sensors module
URL: https://github.com/apache/airflow/pull/5406#discussion_r294081994
##########
File path: airflow/sensors/hdfs_sensor.py
##########
@@ -45,6 +51,13 @@ def __init__(self,
*args,
**kwargs):
super().__init__(*args, **kwargs)
+ if not snakebite_loaded:
+ raise ImportError(
+ 'This HdfsSensor implementation requires snakebite, but '
+ 'snakebite is not compatible with Python 3 '
+ '(as of August 2015). Please use Python 2 if you require '
+ 'this hook -- or help by submitting a PR!')
Review comment:
General question: I always wonder why there's code this short (75 chars)
throughout the code base while the configured line length in Airflow is 110
characters. Is it enforced by some IDE default?
----------------------------------------------------------------
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]
With regards,
Apache Git Services
