airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AIRFLOW-399) Remove dags/testdruid.py
Date Mon, 08 Aug 2016 16:24:20 GMT

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

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

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

[AIRFLOW-399] - Remove dags/testdruid.py

Remove the current example in dags/testdruid.py. It requires the installation of an extraneous
library (Hive) and executes a query against table that no one has. In its place, I am creating
a simple DAG that only depends on DummyOperator and standard Python packages.

Dear Airflow Maintainers,

Please accept this PR that addresses the following issues:
- https://issues.apache.org/jira/browse/AIRFLOW-399

Testing Done:
Manual.

aoen criccomini artwr jlowin bolkedebruin

Closes #1709 from r39132/master


> Remove dags/testdruid.py
> ------------------------
>
>                 Key: AIRFLOW-399
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-399
>             Project: Apache Airflow
>          Issue Type: Bug
>    Affects Versions: Airflow 1.7.1.3
>            Reporter: Siddharth Anand
>            Assignee: Siddharth Anand
>
> Airflow ships with dags/testdruid,py, which is useless. It depends on a hive table that
no-one (but the original committer who set up) has access to. Additionally, it depends on
Hive, which is not a standard package that ships in the setup.py file. Finally, Hive is clearly
not a requirement for using Airflow. 
> Looking over the commit that introduced this, it seems like the committer made a mistake
by committing it -- he used it for his own testing. And, no one corrected this!
> I'm deleting it and adding a simple example dag in its place with no extraneous dependencies



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

Mime
View raw message