You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airflow.apache.org by "Driesprong, Fokko" <fo...@driesprong.frl> on 2019/05/10 10:48:56 UTC

Re: merging of airflow pull request

Sorry for the late reply Ashwini. Could you rebase the PR onto master?
There are issues with Python2.

Cheers, Fokko

Op di 30 apr. 2019 om 22:09 schreef Daniel Imberman <
dimberman.opensource@gmail.com>:

> I'm rerunning it now to see if the failing test will pass. If it does I'll
> merge it.
>
> On Tue, Apr 30, 2019 at 12:31 PM Ashwini Adiga <as...@gmail.com>
> wrote:
>
> > Hi Devs,
> >
> > I would like to know the update on this pull request.
> > https://github.com/apache/airflow/pull/4583,
> > This has been approved by ashb on Mar24 but still not merged. Can you
> > please help this merge the pull request it is very helpful.
> > Thanks in advance for your help.
> >
> > Ashwini
> >
> > On Thu, Feb 28, 2019 at 5:06 PM Ashwini Adiga <as...@gmail.com>
> > wrote:
> >
> > > Hi Devs,
> > >
> > > I would like to get some procedure to merge the pull request:
> > > https://github.com/apache/airflow/pull/4583,
> > > it prevent missing container exit for docker operator with  celery. The
> > >  pull request passes all  the tests and it was reviewed by one
> developer.
> > >
> > > If someone help to get  the procedure to merge the pull request or
> > feedback on the pull request will be off great help.
> > > Thanks in advance for your help.
> > >
> > >
> > > --
> > > Regards,
> > > Ashwini
> > >
> > > ashwini.adigam@gmail.com
> > >
> >
> >
> > --
> > Regards,
> > Ashwini
> > 440-804-6339
> > ashwini.adigam@gmail.com
> >
>