Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-17 Thread Phillip Kelley-Dotson
+1

On Thu, Feb 17, 2022 at 10:31 AM Kamil Gabryjelski <
kamil.gabryjel...@gmail.com> wrote:

> +1
>
> wt., 15 lut 2022, 00:34 użytkownik Aaron Suddjian 
> napisał:
>
> > Hi all,
> >
> > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> > https://github.com/apache/superset/issues/17081
> >
> > The vote will be open for at least 1 week or until the necessary number
> of
> > votes are reached.
> >
> > Please vote accordingly:
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove with the reason
> >
> > Thank you all for your discussion and collaboration around this proposal.
> >
> > --
> > Aaron Suddjian, Software Engineer at Preset
> >
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-17 Thread Kamil Gabryjelski
+1

wt., 15 lut 2022, 00:34 użytkownik Aaron Suddjian  napisał:

> Hi all,
>
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
>
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
>
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thank you all for your discussion and collaboration around this proposal.
>
> --
> Aaron Suddjian, Software Engineer at Preset
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-17 Thread Grace Guo
+1 binding


- grace

> On Feb 14, 2022, at 3:34 PM, Aaron Suddjian  wrote:
> 
> Hi all,
> 
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
> 
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
> 
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
> 
> Thank you all for your discussion and collaboration around this proposal.
> 
> -- 
> Aaron Suddjian, Software Engineer at Preset



Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Hugh Miles
+1


On Mon, Feb 14, 2022 at 3:34 PM, Aaron Suddjian  wrote:

> Hi all,
>
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
>
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
>
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thank you all for your discussion and collaboration around this proposal.
>
> --
> Aaron Suddjian, Software Engineer at Preset
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Maxime Beauchemin
+1 binding

On Tue, 15 Feb 2022 at 09:55, Jiali Kuang  wrote:

> +1 binding
>
> On Tue, Feb 15, 2022 at 6:23 AM Diego Pucci 
> wrote:
>
> > +1
> >
> > Il giorno mar 15 feb 2022 alle ore 15:23 Daniel Gaspar <
> > dpgas...@apache.org>
> > ha scritto:
> >
> > > +1 binding
> > >
> > >
> > > On 2022/02/14 23:34:26 Aaron Suddjian wrote:
> > > > Hi all,
> > > >
> > > > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> > > > https://github.com/apache/superset/issues/17081
> > > >
> > > > The vote will be open for at least 1 week or until the necessary
> number
> > > of
> > > > votes are reached.
> > > >
> > > > Please vote accordingly:
> > > > [ ] +1 approve
> > > > [ ] +0 no opinion
> > > > [ ] -1 disapprove with the reason
> > > >
> > > > Thank you all for your discussion and collaboration around this
> > proposal.
> > > >
> > > > --
> > > > Aaron Suddjian, Software Engineer at Preset
> > > >
> > >
> >
>
>
> --
> *Lily Kuang*
> Full Stack Engineer
> Preset | https://preset.io
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Jiali Kuang
+1 binding

On Tue, Feb 15, 2022 at 6:23 AM Diego Pucci  wrote:

> +1
>
> Il giorno mar 15 feb 2022 alle ore 15:23 Daniel Gaspar <
> dpgas...@apache.org>
> ha scritto:
>
> > +1 binding
> >
> >
> > On 2022/02/14 23:34:26 Aaron Suddjian wrote:
> > > Hi all,
> > >
> > > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> > > https://github.com/apache/superset/issues/17081
> > >
> > > The vote will be open for at least 1 week or until the necessary number
> > of
> > > votes are reached.
> > >
> > > Please vote accordingly:
> > > [ ] +1 approve
> > > [ ] +0 no opinion
> > > [ ] -1 disapprove with the reason
> > >
> > > Thank you all for your discussion and collaboration around this
> proposal.
> > >
> > > --
> > > Aaron Suddjian, Software Engineer at Preset
> > >
> >
>


-- 
*Lily Kuang*
Full Stack Engineer
Preset | https://preset.io


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Diego Pucci
+1

Il giorno mar 15 feb 2022 alle ore 15:23 Daniel Gaspar 
ha scritto:

> +1 binding
>
>
> On 2022/02/14 23:34:26 Aaron Suddjian wrote:
> > Hi all,
> >
> > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> > https://github.com/apache/superset/issues/17081
> >
> > The vote will be open for at least 1 week or until the necessary number
> of
> > votes are reached.
> >
> > Please vote accordingly:
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove with the reason
> >
> > Thank you all for your discussion and collaboration around this proposal.
> >
> > --
> > Aaron Suddjian, Software Engineer at Preset
> >
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Daniel Gaspar
+1 binding


On 2022/02/14 23:34:26 Aaron Suddjian wrote:
> Hi all,
> 
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
> 
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
> 
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
> 
> Thank you all for your discussion and collaboration around this proposal.
> 
> -- 
> Aaron Suddjian, Software Engineer at Preset
> 


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-15 Thread Michael S. Molina
+1 (binding)

> On Feb 15, 2022, at 4:29 AM, Ville Brofeldt  
> wrote:
> 
> +1, really thrilled to finally get rid of some of that old deprecated code. 
> Ville
> 
>> On 15. Feb 2022, at 1.34, Aaron Suddjian  wrote:
>> 
>> Hi all,
>> 
>> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
>> https://github.com/apache/superset/issues/17081
>> 
>> The vote will be open for at least 1 week or until the necessary number of
>> votes are reached.
>> 
>> Please vote accordingly:
>> [ ] +1 approve
>> [ ] +0 no opinion
>> [ ] -1 disapprove with the reason
>> 
>> Thank you all for your discussion and collaboration around this proposal.
>> 
>> -- 
>> Aaron Suddjian, Software Engineer at Preset
> 



Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Ville Brofeldt
+1, really thrilled to finally get rid of some of that old deprecated code. 
Ville

> On 15. Feb 2022, at 1.34, Aaron Suddjian  wrote:
> 
> Hi all,
> 
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
> 
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
> 
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
> 
> Thank you all for your discussion and collaboration around this proposal.
> 
> -- 
> Aaron Suddjian, Software Engineer at Preset



Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Yongjie
+1 binding

On Tue, Feb 15, 2022 at 11:15 AM Tai Dupree  wrote:

> +1 binding
>
> On Mon, Feb 14, 2022 at 6:33 PM Srinivasa Kadamati 
> wrote:
>
> > +1 (binding)
> >
> > This is super exciting!
> >
> > On Mon, Feb 14, 2022 at 8:47 PM John Bodley  > .invalid>
> > wrote:
> >
> > > +1 (binding)
> > >
> > > Thanks Aaron for coordinating this.
> > >
> > > On Tue, Feb 15, 2022 at 2:07 PM Elizabeth Thompson <
> elizab...@preset.io>
> > > wrote:
> > >
> > > > +1 Binding
> > > >
> > > > > On Feb 14, 2022, at 5:02 PM, Aaron Suddjian 
> wrote:
> > > > >
> > > > > In the process of checking the list, I noticed that "Turn on
> > > > > SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong
> column,
> > > and
> > > > > should be considered for inclusion. That's not removal of the
> feature
> > > > flag,
> > > > > just changing its default value.
> > > > >
> > > > > Thanks all, I think we can continue the vote unless there are
> further
> > > > > comments/concerns 
> > > > >
> > > > > On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian 
> > > wrote:
> > > > >
> > > > >> Thanks for pointing that out Beto!
> > > > >>
> > > > >> The VERSIONED_EXPORT item should be thought of as permanently
> > turning
> > > > the
> > > > >> flag on, removing the flag. This is desirable in cases where
> having
> > > the
> > > > >> flag on has become the clearly superior experience, and where
> > > > maintaining
> > > > >> multiple options sustains an undue amount of tech debt. Some other
> > > > flags in
> > > > >> the list got the same treatment.
> > > > >>
> > > > >> Some of us had a meeting to quickly go through the list of
> proposals
> > > > since
> > > > >> there hadn't been much movement in the past months. We decided
> that
> > it
> > > > >> would be good to focus primarily on CLI and config changes. I'll
> > post
> > > > >> explanations for the ones considered unsuitable on the SIP.
> > > > >>
> > > > >> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
> > > > >>  wrote:
> > > > >>
> > > > >>> I have a comment on one item:
> > > > >>>
> > > > >>> # Remove old import/export code and turn on the VERSIONED_EXPORT
> > flag
> > > > by
> > > > >>> default.
> > > > >>>
> > > > >>> We can't remove the old code and have the feature flag at the
> same
> > > > time;
> > > > >>> the old code is needed to turn the feature flag off.
> > > > >>>
> > > > >>> It also seems that many features were considered unsuitable
> > without a
> > > > >>> comment explaining why.
> > > > >>>
> > > > >>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
> > > >  Hi all,
> > > > 
> > > >  I'd like to call for a vote on SIP-74, Superset 2.0 Breaking
> > > Changes:
> > > >  https://github.com/apache/superset/issues/17081
> > > > 
> > > >  The vote will be open for at least 1 week or until the necessary
> > > > number
> > > > >>> of
> > > >  votes are reached.
> > > > 
> > > >  Please vote accordingly:
> > > >  [ ] +1 approve
> > > >  [ ] +0 no opinion
> > > >  [ ] -1 disapprove with the reason
> > > > 
> > > >  Thank you all for your discussion and collaboration around this
> > > > >>> proposal.
> > > > 
> > > > >>>
> > > > >>
> > > > >>
> > > > >> --
> > > > >> Aaron Suddjian, Software Engineer at Preset
> > > > >>
> > > > >
> > > > >
> > > > > --
> > > > > Aaron Suddjian, Software Engineer at Preset
> > > >
> > > >
> > >
> >
>
>
> --
> -Tai Dupree
>


-- 

Best regards,

Yongjie


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Tai Dupree
+1 binding

On Mon, Feb 14, 2022 at 6:33 PM Srinivasa Kadamati  wrote:

> +1 (binding)
>
> This is super exciting!
>
> On Mon, Feb 14, 2022 at 8:47 PM John Bodley  .invalid>
> wrote:
>
> > +1 (binding)
> >
> > Thanks Aaron for coordinating this.
> >
> > On Tue, Feb 15, 2022 at 2:07 PM Elizabeth Thompson 
> > wrote:
> >
> > > +1 Binding
> > >
> > > > On Feb 14, 2022, at 5:02 PM, Aaron Suddjian  wrote:
> > > >
> > > > In the process of checking the list, I noticed that "Turn on
> > > > SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong column,
> > and
> > > > should be considered for inclusion. That's not removal of the feature
> > > flag,
> > > > just changing its default value.
> > > >
> > > > Thanks all, I think we can continue the vote unless there are further
> > > > comments/concerns 
> > > >
> > > > On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian 
> > wrote:
> > > >
> > > >> Thanks for pointing that out Beto!
> > > >>
> > > >> The VERSIONED_EXPORT item should be thought of as permanently
> turning
> > > the
> > > >> flag on, removing the flag. This is desirable in cases where having
> > the
> > > >> flag on has become the clearly superior experience, and where
> > > maintaining
> > > >> multiple options sustains an undue amount of tech debt. Some other
> > > flags in
> > > >> the list got the same treatment.
> > > >>
> > > >> Some of us had a meeting to quickly go through the list of proposals
> > > since
> > > >> there hadn't been much movement in the past months. We decided that
> it
> > > >> would be good to focus primarily on CLI and config changes. I'll
> post
> > > >> explanations for the ones considered unsuitable on the SIP.
> > > >>
> > > >> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
> > > >>  wrote:
> > > >>
> > > >>> I have a comment on one item:
> > > >>>
> > > >>> # Remove old import/export code and turn on the VERSIONED_EXPORT
> flag
> > > by
> > > >>> default.
> > > >>>
> > > >>> We can't remove the old code and have the feature flag at the same
> > > time;
> > > >>> the old code is needed to turn the feature flag off.
> > > >>>
> > > >>> It also seems that many features were considered unsuitable
> without a
> > > >>> comment explaining why.
> > > >>>
> > > >>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
> > >  Hi all,
> > > 
> > >  I'd like to call for a vote on SIP-74, Superset 2.0 Breaking
> > Changes:
> > >  https://github.com/apache/superset/issues/17081
> > > 
> > >  The vote will be open for at least 1 week or until the necessary
> > > number
> > > >>> of
> > >  votes are reached.
> > > 
> > >  Please vote accordingly:
> > >  [ ] +1 approve
> > >  [ ] +0 no opinion
> > >  [ ] -1 disapprove with the reason
> > > 
> > >  Thank you all for your discussion and collaboration around this
> > > >>> proposal.
> > > 
> > > >>>
> > > >>
> > > >>
> > > >> --
> > > >> Aaron Suddjian, Software Engineer at Preset
> > > >>
> > > >
> > > >
> > > > --
> > > > Aaron Suddjian, Software Engineer at Preset
> > >
> > >
> >
>


-- 
-Tai Dupree


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Srinivasa Kadamati
+1 (binding)

This is super exciting!

On Mon, Feb 14, 2022 at 8:47 PM John Bodley 
wrote:

> +1 (binding)
>
> Thanks Aaron for coordinating this.
>
> On Tue, Feb 15, 2022 at 2:07 PM Elizabeth Thompson 
> wrote:
>
> > +1 Binding
> >
> > > On Feb 14, 2022, at 5:02 PM, Aaron Suddjian  wrote:
> > >
> > > In the process of checking the list, I noticed that "Turn on
> > > SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong column,
> and
> > > should be considered for inclusion. That's not removal of the feature
> > flag,
> > > just changing its default value.
> > >
> > > Thanks all, I think we can continue the vote unless there are further
> > > comments/concerns 
> > >
> > > On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian 
> wrote:
> > >
> > >> Thanks for pointing that out Beto!
> > >>
> > >> The VERSIONED_EXPORT item should be thought of as permanently turning
> > the
> > >> flag on, removing the flag. This is desirable in cases where having
> the
> > >> flag on has become the clearly superior experience, and where
> > maintaining
> > >> multiple options sustains an undue amount of tech debt. Some other
> > flags in
> > >> the list got the same treatment.
> > >>
> > >> Some of us had a meeting to quickly go through the list of proposals
> > since
> > >> there hadn't been much movement in the past months. We decided that it
> > >> would be good to focus primarily on CLI and config changes. I'll post
> > >> explanations for the ones considered unsuitable on the SIP.
> > >>
> > >> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
> > >>  wrote:
> > >>
> > >>> I have a comment on one item:
> > >>>
> > >>> # Remove old import/export code and turn on the VERSIONED_EXPORT flag
> > by
> > >>> default.
> > >>>
> > >>> We can't remove the old code and have the feature flag at the same
> > time;
> > >>> the old code is needed to turn the feature flag off.
> > >>>
> > >>> It also seems that many features were considered unsuitable without a
> > >>> comment explaining why.
> > >>>
> > >>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
> >  Hi all,
> > 
> >  I'd like to call for a vote on SIP-74, Superset 2.0 Breaking
> Changes:
> >  https://github.com/apache/superset/issues/17081
> > 
> >  The vote will be open for at least 1 week or until the necessary
> > number
> > >>> of
> >  votes are reached.
> > 
> >  Please vote accordingly:
> >  [ ] +1 approve
> >  [ ] +0 no opinion
> >  [ ] -1 disapprove with the reason
> > 
> >  Thank you all for your discussion and collaboration around this
> > >>> proposal.
> > 
> > >>>
> > >>
> > >>
> > >> --
> > >> Aaron Suddjian, Software Engineer at Preset
> > >>
> > >
> > >
> > > --
> > > Aaron Suddjian, Software Engineer at Preset
> >
> >
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread John Bodley
+1 (binding)

Thanks Aaron for coordinating this.

On Tue, Feb 15, 2022 at 2:07 PM Elizabeth Thompson 
wrote:

> +1 Binding
>
> > On Feb 14, 2022, at 5:02 PM, Aaron Suddjian  wrote:
> >
> > In the process of checking the list, I noticed that "Turn on
> > SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong column, and
> > should be considered for inclusion. That's not removal of the feature
> flag,
> > just changing its default value.
> >
> > Thanks all, I think we can continue the vote unless there are further
> > comments/concerns 
> >
> > On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian  wrote:
> >
> >> Thanks for pointing that out Beto!
> >>
> >> The VERSIONED_EXPORT item should be thought of as permanently turning
> the
> >> flag on, removing the flag. This is desirable in cases where having the
> >> flag on has become the clearly superior experience, and where
> maintaining
> >> multiple options sustains an undue amount of tech debt. Some other
> flags in
> >> the list got the same treatment.
> >>
> >> Some of us had a meeting to quickly go through the list of proposals
> since
> >> there hadn't been much movement in the past months. We decided that it
> >> would be good to focus primarily on CLI and config changes. I'll post
> >> explanations for the ones considered unsuitable on the SIP.
> >>
> >> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
> >>  wrote:
> >>
> >>> I have a comment on one item:
> >>>
> >>> # Remove old import/export code and turn on the VERSIONED_EXPORT flag
> by
> >>> default.
> >>>
> >>> We can't remove the old code and have the feature flag at the same
> time;
> >>> the old code is needed to turn the feature flag off.
> >>>
> >>> It also seems that many features were considered unsuitable without a
> >>> comment explaining why.
> >>>
> >>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
>  Hi all,
> 
>  I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
>  https://github.com/apache/superset/issues/17081
> 
>  The vote will be open for at least 1 week or until the necessary
> number
> >>> of
>  votes are reached.
> 
>  Please vote accordingly:
>  [ ] +1 approve
>  [ ] +0 no opinion
>  [ ] -1 disapprove with the reason
> 
>  Thank you all for your discussion and collaboration around this
> >>> proposal.
> 
> >>>
> >>
> >>
> >> --
> >> Aaron Suddjian, Software Engineer at Preset
> >>
> >
> >
> > --
> > Aaron Suddjian, Software Engineer at Preset
>
>


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Elizabeth Thompson
+1 Binding 

> On Feb 14, 2022, at 5:02 PM, Aaron Suddjian  wrote:
> 
> In the process of checking the list, I noticed that "Turn on
> SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong column, and
> should be considered for inclusion. That's not removal of the feature flag,
> just changing its default value.
> 
> Thanks all, I think we can continue the vote unless there are further
> comments/concerns 
> 
> On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian  wrote:
> 
>> Thanks for pointing that out Beto!
>> 
>> The VERSIONED_EXPORT item should be thought of as permanently turning the
>> flag on, removing the flag. This is desirable in cases where having the
>> flag on has become the clearly superior experience, and where maintaining
>> multiple options sustains an undue amount of tech debt. Some other flags in
>> the list got the same treatment.
>> 
>> Some of us had a meeting to quickly go through the list of proposals since
>> there hadn't been much movement in the past months. We decided that it
>> would be good to focus primarily on CLI and config changes. I'll post
>> explanations for the ones considered unsuitable on the SIP.
>> 
>> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
>>  wrote:
>> 
>>> I have a comment on one item:
>>> 
>>> # Remove old import/export code and turn on the VERSIONED_EXPORT flag by
>>> default.
>>> 
>>> We can't remove the old code and have the feature flag at the same time;
>>> the old code is needed to turn the feature flag off.
>>> 
>>> It also seems that many features were considered unsuitable without a
>>> comment explaining why.
>>> 
>>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
 Hi all,
 
 I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
 https://github.com/apache/superset/issues/17081
 
 The vote will be open for at least 1 week or until the necessary number
>>> of
 votes are reached.
 
 Please vote accordingly:
 [ ] +1 approve
 [ ] +0 no opinion
 [ ] -1 disapprove with the reason
 
 Thank you all for your discussion and collaboration around this
>>> proposal.
 
>>> 
>> 
>> 
>> --
>> Aaron Suddjian, Software Engineer at Preset
>> 
> 
> 
> -- 
> Aaron Suddjian, Software Engineer at Preset



Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Aaron Suddjian
In the process of checking the list, I noticed that "Turn on
SQLLAB_BACKEND_PERSISTENCE by default" ended up in the wrong column, and
should be considered for inclusion. That's not removal of the feature flag,
just changing its default value.

Thanks all, I think we can continue the vote unless there are further
comments/concerns 

On Mon, Feb 14, 2022 at 4:36 PM Aaron Suddjian  wrote:

> Thanks for pointing that out Beto!
>
> The VERSIONED_EXPORT item should be thought of as permanently turning the
> flag on, removing the flag. This is desirable in cases where having the
> flag on has become the clearly superior experience, and where maintaining
> multiple options sustains an undue amount of tech debt. Some other flags in
> the list got the same treatment.
>
> Some of us had a meeting to quickly go through the list of proposals since
> there hadn't been much movement in the past months. We decided that it
> would be good to focus primarily on CLI and config changes. I'll post
> explanations for the ones considered unsuitable on the SIP.
>
> On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
>  wrote:
>
>> I have a comment on one item:
>>
>> # Remove old import/export code and turn on the VERSIONED_EXPORT flag by
>> default.
>>
>> We can't remove the old code and have the feature flag at the same time;
>> the old code is needed to turn the feature flag off.
>>
>> It also seems that many features were considered unsuitable without a
>> comment explaining why.
>>
>> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
>> > Hi all,
>> >
>> > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
>> > https://github.com/apache/superset/issues/17081
>> >
>> > The vote will be open for at least 1 week or until the necessary number
>> of
>> > votes are reached.
>> >
>> > Please vote accordingly:
>> > [ ] +1 approve
>> > [ ] +0 no opinion
>> > [ ] -1 disapprove with the reason
>> >
>> > Thank you all for your discussion and collaboration around this
>> proposal.
>> >
>>
>
>
> --
> Aaron Suddjian, Software Engineer at Preset
>


-- 
Aaron Suddjian, Software Engineer at Preset


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Evan Rusackas
+1, binding!

Been waiting a long time for this, and super excited to see it gaining 
momentum. This release will also help us get better at planning and releasing 
major versions, supporting LTS branches, etc. Getting to 2.0 is exciting 
enough, but I’m equally excited about being able to march toward 3.0 and beyond.

Thanks,

-e-

Evan Rusackas
Preset | preset.io
On Feb 14, 2022, 4:34 PM -0700, Aaron Suddjian , wrote:
> Hi all,
>
> I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> https://github.com/apache/superset/issues/17081
>
> The vote will be open for at least 1 week or until the necessary number of
> votes are reached.
>
> Please vote accordingly:
> [ ] +1 approve
> [ ] +0 no opinion
> [ ] -1 disapprove with the reason
>
> Thank you all for your discussion and collaboration around this proposal.
>
> --
> Aaron Suddjian, Software Engineer at Preset


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Beto Dealmeida
Ah, makes sense. So we're getting rid of the feature flag — I thought we 
were just changing the default value.


+1, binding.

On 2/14/22 4:36 PM, Aaron Suddjian wrote:

Thanks for pointing that out Beto!

The VERSIONED_EXPORT item should be thought of as permanently turning the
flag on, removing the flag. This is desirable in cases where having the
flag on has become the clearly superior experience, and where maintaining
multiple options sustains an undue amount of tech debt. Some other flags in
the list got the same treatment.

Some of us had a meeting to quickly go through the list of proposals since
there hadn't been much movement in the past months. We decided that it
would be good to focus primarily on CLI and config changes. I'll post
explanations for the ones considered unsuitable on the SIP.

On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
 wrote:


I have a comment on one item:

# Remove old import/export code and turn on the VERSIONED_EXPORT flag by
default.

We can't remove the old code and have the feature flag at the same time;
the old code is needed to turn the feature flag off.

It also seems that many features were considered unsuitable without a
comment explaining why.

On 2/14/22 3:34 PM, Aaron Suddjian wrote:

Hi all,

I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
https://github.com/apache/superset/issues/17081

The vote will be open for at least 1 week or until the necessary number

of

votes are reached.

Please vote accordingly:
[ ] +1 approve
[ ] +0 no opinion
[ ] -1 disapprove with the reason

Thank you all for your discussion and collaboration around this proposal.







Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Aaron Suddjian
Thanks for pointing that out Beto!

The VERSIONED_EXPORT item should be thought of as permanently turning the
flag on, removing the flag. This is desirable in cases where having the
flag on has become the clearly superior experience, and where maintaining
multiple options sustains an undue amount of tech debt. Some other flags in
the list got the same treatment.

Some of us had a meeting to quickly go through the list of proposals since
there hadn't been much movement in the past months. We decided that it
would be good to focus primarily on CLI and config changes. I'll post
explanations for the ones considered unsuitable on the SIP.

On Mon, Feb 14, 2022 at 3:41 PM Beto Dealmeida
 wrote:

> I have a comment on one item:
>
> # Remove old import/export code and turn on the VERSIONED_EXPORT flag by
> default.
>
> We can't remove the old code and have the feature flag at the same time;
> the old code is needed to turn the feature flag off.
>
> It also seems that many features were considered unsuitable without a
> comment explaining why.
>
> On 2/14/22 3:34 PM, Aaron Suddjian wrote:
> > Hi all,
> >
> > I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
> > https://github.com/apache/superset/issues/17081
> >
> > The vote will be open for at least 1 week or until the necessary number
> of
> > votes are reached.
> >
> > Please vote accordingly:
> > [ ] +1 approve
> > [ ] +0 no opinion
> > [ ] -1 disapprove with the reason
> >
> > Thank you all for your discussion and collaboration around this proposal.
> >
>


-- 
Aaron Suddjian, Software Engineer at Preset


Re: [VOTE][SIP-74] Breaking Changes to include in Superset 2.0

2022-02-14 Thread Beto Dealmeida

I have a comment on one item:

# Remove old import/export code and turn on the VERSIONED_EXPORT flag by 
default.


We can't remove the old code and have the feature flag at the same time; 
the old code is needed to turn the feature flag off.


It also seems that many features were considered unsuitable without a 
comment explaining why.


On 2/14/22 3:34 PM, Aaron Suddjian wrote:

Hi all,

I'd like to call for a vote on SIP-74, Superset 2.0 Breaking Changes:
https://github.com/apache/superset/issues/17081

The vote will be open for at least 1 week or until the necessary number of
votes are reached.

Please vote accordingly:
[ ] +1 approve
[ ] +0 no opinion
[ ] -1 disapprove with the reason

Thank you all for your discussion and collaboration around this proposal.