Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-16 Thread Bolke de Bruin
PR is done and ready for proper review. Please note: * The recursive has been removed. It made no sense in my opinion. SubDags and their tasks are always considered. * It is dependent on existing dag runs. You cannot add a task instance somewhere in the middle of no-where. * It will create dag

Re: [RESULT] [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-16 Thread Bolke de Bruin
Problems solved. PR can be reviewed. Cheers Bolke > On 16 Feb 2017, at 22:14, Bolke de Bruin wrote: > > Hi, > > Just a small update. I have fixed the issue with mark success in PR > (https://github.com/apache/incubator-airflow/pull/2085 >

Re: [RESULT] [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-16 Thread Bolke de Bruin
Hi, Just a small update. I have fixed the issue with mark success in PR (https://github.com/apache/incubator-airflow/pull/2085 ), but I am struggling with getting Travis’ tests to pass (locally they’re fine). Bolke > On 13 Feb 2017, at

Re: [RESULT] [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-13 Thread Bolke de Bruin
Thanks! I am holding off the IPMC until we resolved the “mark success” issue. I feel (and others with me) too many are reliant on this feature. I am reworking the functionality, but it will take me at least another day unfortunately. - Bolke > On 13 Feb 2017, at 22:20, Maxime Beauchemin

Re: [RESULT] [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-13 Thread Maxime Beauchemin
Infinitely grateful for your work! It's a nice demonstration on how one person can move mountains. Thanks Bolke! Max On Mon, Feb 13, 2017 at 1:06 PM, Chris Riccomini wrote: > Hey Bolke, > > Fantastic work. Thanks so much for your effort--it's a huge service to the >

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-13 Thread Bolke de Bruin
You raise a good point on the “extra_task” thing. I havent tested that. I am discussing matters on gitter with Sid as well (although he is in a meeting now). I am holding off raising the vote on the IPMC (*sigh* ;) ). Bolke > On 13 Feb 2017, at 21:40, Dan Davydov

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-13 Thread Bolke de Bruin
https://github.com/apache/incubator-airflow/pull/2075 Is (part of) the fix. I can include it retroactively if needed, but I don’t consider it blocking. Bolke > On 13 Feb 2017, at 20:56, Dan Davydov

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-13 Thread Alex Van Boxel
+1 (binding) On Mon, Feb 13, 2017 at 7:45 PM siddharth anand wrote: > +1 (binding) > > On Mon, Feb 13, 2017 at 8:57 AM, Chris Riccomini > wrote: > > > +1 (binding) > > > > On Sun, Feb 12, 2017 at 8:54 AM, Jeremiah Lowin > wrote: > >

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-11 Thread siddharth anand
Deployed to stage and will watch over the weekend before voting. -s On Sat, Feb 11, 2017 at 9:53 AM, Jeremiah Lowin wrote: > Boris, I submitted a PR to address your second point -- > https://github.com/apache/incubator-airflow/pull/2068. Thanks! > > On Sat, Feb 11, 2017 at

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-11 Thread Boris Tyukin
I am running LocalExecutor and not doing crazy things but use DAG generation heavily - everything runs fine as before. As I mentioned in other threads only had a few issues: 1) had to upgrade MySQL which was a PAIN. Cloudera CDH is running old version of MySQL which was compatible with 1.7.1 but

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-10 Thread Maxime Beauchemin
+1 (binding) On Fri, Feb 10, 2017 at 3:44 PM, Arthur Wiedmer wrote: > +1 (binding) > > On Feb 10, 2017 3:13 PM, "Dan Davydov" > wrote: > > > Our staging looks good, all the DAGs there pass. > > +1 (binding) > > > > On Fri, Feb 10, 2017

Re: [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc3

2017-02-10 Thread Dan Davydov
Our staging looks good, all the DAGs there pass. +1 (binding) On Fri, Feb 10, 2017 at 10:21 AM, Chris Riccomini wrote: > Running in all environments. Will vote after the weekend to make sure > things are working properly, but so far so good. > > On Fri, Feb 10, 2017 at