Let me cancel the vote and start it for all of them. As discussed in the separate thread - all three of the "to-be-moved" executors already have their own extra (and in our docs it was specified that they are needed for executors), and those extras are needed anyway (they bring the right dependencies and quite many of them). For example I think if someone previously used celery and did NOT use the "celery" extra and used Celery Executor (same with k8) - it would have to be really special way of installing airflow (install airflow + install all celery deps individually) - this is already custom enough and "special enough" to a fall into https://www.hyrumslaw.com/ squarely.
J. On Fri, Jul 21, 2023 at 7:58 PM Oliveira, Niko <[email protected]> wrote: > > -1 (binding) > > I think the eventual goal is all 3rd party executors (excluding Local, > Sequential, etc) are not pre-installed. I think it will take a while for us > to get there with Celery and K8s but it's the right thing to shoot for and we > should start with Dask. > > ________________________________ > From: Collin McNulty <[email protected]> > Sent: Friday, July 21, 2023 8:37:38 AM > To: [email protected] > Subject: RE: [EXTERNAL][VOTE] Make (soon coming) dask provider preinstalled > > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you can confirm the sender and know the > content is safe. > > > > -1 (non-binding) > I agree with Ash’s reasoning. > -- > > Collin McNulty > Director of Global Support > > Email: [email protected] <[email protected]> > Time zone: US Central (CST UTC-6 / CDT UTC-5) > > > <https://www.astronomer.io/> --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
