+1 (non-binding) :)
On Fri, Apr 24, 2020 at 10:00 AM Xinbin Huang <bin.huan...@gmail.com> wrote: > +1 non-binding > > Best, > Bin > > On Fri, Apr 24, 2020 at 9:20 AM Tomasz Urbaszek < > tomasz.urbas...@polidea.com> > wrote: > > > +1 binding > > > > Tomek > > > > > > On Fri, Apr 24, 2020 at 6:15 PM QP Hou <q...@scribd.com> wrote: > > > > > > +1 (non binding) > > > > > > On Fri, Apr 24, 2020 at 3:31 AM Ash Berlin-Taylor <a...@apache.org> > > wrote: > > > > > > > > As we discussed in the thread on Spec thread for our new API, the > idea > > > > came up of making conn_id unique in Airflow > > > > > > > > < > > > https://lists.apache.org/thread.html/rfb6f7d95b2754fda9dd09b08444214dfad12d10f143d32de0fcf4104%40%3Cdev.airflow.apache.org%3E > > > > > > > > > > > The people in that thread seemed to be in favour of it: > > > > > > > > - It is often confusing to users (there have been a few bug reports > > > > about it over the years) > > > > - It's questionable if it actually works well or not > > > > - There are better/smarter tools for loadbalancing connections to a > DB > > > > than picking one of a random list > > > > - For Hive at least it has been implemented another way - allowing > two > > > > host's in a single connection string > > > > https://github.com/apache/airflow/pull/4708 (available in 1.10.6) > > > > - It makes the HTTP API confusing (needing a integer connection ID, > and > > > > a string conn_id field) > > > > > > > > Given the downsides/work arounds, and the confusion this causes to I > > > > propose we remove this (mis)feature from Airflow 2.0. > > > > > > > > Kevin from AirBnb (on bcc, hopefully you see this) I would be > > interested > > > > in your vote/view, as I know AirBnB was using this in the past. > > > > > > > > This vote will last for 96 Hours, until 10:30Z on 2020-04-27 > > https://www.timeanddate.com/worldclock/fixedtime.html?iso=20200427T1030 > > > > > > > > Every vote counts here, so please all vote. > > > > > > > > Options: > > > > > > > > > > > > [ ] +1 approve > > > > [ ] +0 no opinion > > > > [ ] -1 disapprove with the reason > > > > > > > > (or fractional values between -1..+1) > > > > > > > > This is my +1 vote. > > > > > > > > Thanks, > > > > Ash > > > > > > > > -- > > > > Tomasz Urbaszek > > Polidea | Software Engineer > > > > M: +48 505 628 493 > > E: tomasz.urbas...@polidea.com > > > > Unique Tech > > Check out our projects! > > >
