Mailing-List: contact commits-help@airflow.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@airflow.incubator.apache.org
Date: Mon, 8 Aug 2016 16:24:20 +0000 (UTC)
From: "ASF subversion and git services (JIRA)" <jira@apache.org>
To: commits@airflow.incubator.apache.org
Message-ID: <JIRA.12995106.1470376107000.250196.1470673460710@Atlassian.JIRA>
In-Reply-To: <JIRA.12995106.1470376107000@Atlassian.JIRA>
References: <JIRA.12995106.1470376107000@Atlassian.JIRA> <JIRA.12995106.1470376107416@arcas>
Subject: [jira] [Commented] (AIRFLOW-399) Remove dags/testdruid.py
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Mon, 08 Aug 2016 16:24:26 -0000


    [ 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)