airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <>
Subject [jira] [Commented] (AIRFLOW-5640) BaseOperator email parameter is wrongly typed and not documented
Date Mon, 16 Dec 2019 11:19:00 GMT


ASF subversion and git services commented on AIRFLOW-5640:

Commit b9ba7191329144a8d7a5441288a2db21f0699a80 in airflow's branch refs/heads/v1-10-test
from Martin Winkel
[;h=b9ba719 ]

[AIRFLOW-5640] Document and test `email` parameters of BaseOperator (#6315)

* Refactored get_email_address_list to have a better
  separation between string handling and other iterables.
* Explicitely casting get_email_address_list argument
  to a list in case the argument was an iterable. This
  enables direct support for tuples, sets or the like.
* Fixed type annotation of email parameter of
  BaseOperator to show that iterables are directly
* Added docstring entries for email, email_on_retry,
  email_on_failure and queue in BaseOperator.

(cherry picked from commit 9ec562f88ef8e690f0b17526878b46847f0422e7)

> BaseOperator email parameter is wrongly typed and not documented
> ----------------------------------------------------------------
>                 Key: AIRFLOW-5640
>                 URL:
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: operators
>    Affects Versions: 1.10.5
>            Reporter: Cedrik Neumann
>            Assignee: Martin Winkel
>            Priority: Major
>             Fix For: 1.10.7
> The {{email}} field is not documented in BaseOperator and furthermore the type annotation
{{str}} is wrong [here|].
> The method {{get_email_address_list}} clearly accepts lists of strings as well as comma
and semicolon delimited lists: [here|]

This message was sent by Atlassian Jira

View raw message