msumit commented on a change in pull request #6709: [AIRFLOW-5834] Option to 
skip serve_logs process with workers
URL: https://github.com/apache/airflow/pull/6709#discussion_r353080350
 
 

 ##########
 File path: pylintrc
 ##########
 @@ -154,6 +154,8 @@ disable=print-statement,
         deprecated-sys-function,
         exception-escape,
         comprehension-escape,
+        too-many-arguments,
 
 Review comment:
   @aoen I think it's doing exactly the same. It is not removing coverage of 
all such occurrences, instead it is just a while list of skippable offenses. 
For ex these were the errors I was getting before
   ```
   airflow/contrib/operators/hive_to_dynamodb.py:65:    def __init__(  # 
pylint:disable=too-many-arguments
   airflow/operators/hive_to_druid.py:87:    def __init__(  # 
pylint:disable=too-many-arguments
   airflow/operators/mysql_to_hive.py:88:    def __init__(  # 
pylint:disable=too-many-arguments
   airflow/operators/s3_to_hive_operator.py:40:class 
S3ToHiveTransfer(BaseOperator):  # pylint:disable=too-many-instance-attributes
   airflow/operators/s3_to_hive_operator.py:111:    def __init__(  # 
pylint:disable=too-many-arguments
   ```
   
   So u can see that even #pylint was mentioned the error was being thrown. 

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to