airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xiaodong DENG (JIRA)" <>
Subject [jira] [Resolved] (AIRFLOW-2836) Minor improvement of contrib.sensors.FileSensor
Date Wed, 08 Aug 2018 02:01:00 GMT


Xiaodong DENG resolved AIRFLOW-2836.
    Resolution: Fixed

> Minor improvement of contrib.sensors.FileSensor
> -----------------------------------------------
>                 Key: AIRFLOW-2836
>                 URL:
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: contrib
>            Reporter: Xiaodong DENG
>            Assignee: Xiaodong DENG
>            Priority: Minor
> h4. *Background*
> The default *fs_conn_id* in contrib.sensors.FileSensor is '_*fs_default2*_'. However,
when we initiate the database (,
there isn't such an entry. It doesn't exist anywhere else.
> h4. *Issue*
> The purpose of _contrib.sensors.FileSensor_ is mainly for checking local file system
(of course can also be used for NAS). Then the path ("/") from default connection 'fs_default'
would suffice.
> However, given the default value for *fs_conn_id* in contrib.sensors.FileSensor is
"fs_default2" (a value doesn't exist), it will make the situation much more complex. 
> When users intend to check local file system only, they should be able to leave *fs_conn_id* default
directly, instead of going setting up another connection separately.
> h4. Proposal
> Change default value for *fs_conn_id* in contrib.sensors.FileSensor from "fs_default2"
to "fs_default" (actually in the related test, the *fs_conn_id* are all specified to be "fs_default").

This message was sent by Atlassian JIRA

View raw message