Good idea overall.
I am also aligned towards moving it out of `db` subcommand.  I am not very
much aligned towards using the keyword/action word - `upgrade`. It sets a
different context.

Thanks,
Amogh Desai

On Thu, Jul 27, 2023 at 1:02 AM Jarek Potiuk <[email protected]> wrote:

> +1 for moving it under "connections" - good idea.
>
> I think "upgrade" is confusing for the first time because upgrade
> implies "upgrade FROM something".  The "sync" is not set-in-stone.
> Maybe there are other ideas?
>
> The inspiration is Terraform - where we describe the "target" state we
> want to reach, so maybe "airflow db apply" is a better one?
>
> J.
>
> On Wed, Jul 26, 2023 at 8:51 PM Jed Cunningham <[email protected]>
> wrote:
> >
> > I'm not sure I like sync over upgrade, personally. Everything else
> > discussed sounds good though.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

Reply via email to