airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick McKenna (JIRA)" <j...@apache.org>
Subject [jira] [Created] (AIRFLOW-814) Presto*CheckOperator should use keyword args during initialization
Date Fri, 27 Jan 2017 23:48:25 GMT
Patrick McKenna created AIRFLOW-814:
---------------------------------------

             Summary: Presto*CheckOperator should use keyword args during initialization
                 Key: AIRFLOW-814
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-814
             Project: Apache Airflow
          Issue Type: Bug
            Reporter: Patrick McKenna
            Priority: Trivial


PrestoValueCheckOperator and PrestoIntervalCheckOperator don't forward user-supplied args
as keyword args to their parent class initializers, which causes an exception to be raised
by the apply_defaults decorator (see https://github.com/apache/incubator-airflow/blob/cf6d50c0e3d59ec9155be445c127c90a05cc90a2/airflow/utils/decorators.py#L46).

For example:

>>> from airflow.operators import PrestoIntervalCheckOperator
>>> p = PrestoIntervalCheckOperator(task_id='id', table='table', metrics_thresholds=0)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/.venv/lib/python3.5/site-packages/airflow/utils/decorators.py", line 86, in wrapper
    result = func(*args, **kwargs)
  File "/.venv/lib/python3.5/site-packages/airflow/operators/presto_check_operator.py", line
99, in __init__
    *args, **kwargs)
  File "/.venv/lib/python3.5/site-packages/airflow/utils/decorators.py", line 46, in wrapper
    "Use keyword arguments when initializing operators")
airflow.exceptions.AirflowException: Use keyword arguments when initializing operators



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message