Gaetan,

I did a quick review. Overall the changes look good. A couple of things to
check.

Best,
Arthur

On Sat, Jan 6, 2018 at 1:46 AM, Gaetan Semet <[email protected]> wrote:

> Hello
>
> Can a core developer look at my pull request ?
> https://github.com/apache/incubator-airflow/pull/2723
>
> Thanks
> Gaetan
>
> On 2017-12-10 14:37, Sergei Iakhnin <[email protected]> wrote:
> > This feature sounds very useful. I hope you get your PR in soon.
> >
> > On Sat, Dec 9, 2017 at 7:59 PM Gaetan Semet <[email protected]> wrote:
> >
> > > Hello
> > >
> > > Sorry to insist on my PR review, but it seems it can help some people:
> > > https://github.com/apache/incubator-airflow/pull/2723#
> issuecomment-350356492
> > >
> > > For recall, this allows to easily set up airflow as url such as
> > >
> > > https://lab.mycompany.com/myorg/airflow/
> > >
> > > and flower in
> > >
> > > https://lab.mycompany.com/myorg/airflow/flower
> > >
> > > API will be accessible at
> > >
> > > https://lab.mycompany.com/myorg/airflow/api/experimental/...
> > >
> > > Thanks
> > >
> > > Gaetan Semet
> > >
> > --
> >
> > Sergei
> >
>

Reply via email to