[ 
https://issues.apache.org/jira/browse/AIRFLOW-2147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16383345#comment-16383345
 ] 

ASF subversion and git services commented on AIRFLOW-2147:
----------------------------------------------------------

Commit fedc5a092c4ceb74c7d02ff932ad7de796705d43 in incubator-airflow's branch 
refs/heads/master from [~arcward]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-airflow.git;h=fedc5a0 ]

[AIRFLOW-2147] Plugin manager: added 'sensors' attribute

AirflowPlugin required both BaseOperator and BaseSensorOperator
to be included in its `operators` attribute. Add a `sensors`
attribute and updated import logic so that anything added to
the new attribute can be imported from `airflow.sensors.{plugin_name}`

The integration/`make_module` calls in `airflow.plugins_manager`
for operators is also updated to maintain the ability to import
sensors from `operators` to avoid breaking existing plugins

- Update unit tests and documentation to reflect this
- Added exclusion for flake8 module level import not at top of file

Closes #3075 from arcward/AIRFLOW-2147


> Add 'sensors' attribute for plugins
> -----------------------------------
>
>                 Key: AIRFLOW-2147
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-2147
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: plugins
>    Affects Versions: Airflow 1.8
>            Reporter: Edward Wells
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 2.0.0
>
>
> I created a pull request: 
> https://github.com/apache/incubator-airflow/pull/3075
> {{airflow.plugins_manager.AirflowPlugin}} has attributes for operators, 
> hooks, executors, macros, admin_views, flask_blueprints and menu_links, but 
> not for sensors.
> Right now, operators/sensors can be imported like:
> {{    from airflow.operators.test_plugin import PluginOperator}}
> It would be intuitive to have sensors importable similarly like:
> {{    from airflow.sensors.test_plugin import PluginSensorOperator}} 
> The first time I wrote a plugin I instinctively did it this way until I 
> realized I had to bundle both operators and sensors into operators



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to