Thanks Gerardo! I think now we need to discuss Github Actions vs. CWiki :). Happy to do start it when I am back from vacation (next week).
J. On Mon, Aug 31, 2020 at 6:52 PM Gerard Casas Saez <gcasass...@twitter.com.invalid> wrote: > As I mentioned in my previous email, I assume a lazy consensus for this. > Merged the latest proposal in doc: > > https://docs.google.com/document/d/1hJDodc1xl4ixswPzAV4IAQu74sLnOOKjqecPgthLta4/edit# > to > CWIKI page: > > https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals > > Formatting wise, it seems it got formatted nicely just copy pasting so I > don't think I will modify this. > > Thanks everyone for your contributions! > > Best, > > Gerard Casas Saez > Twitter | Cortex | @casassaez <http://twitter.com/casassaez> > > > On Thu, Aug 27, 2020 at 12:08 PM Kaxil Naik <kaxiln...@gmail.com> wrote: > > > Yes go for it > > > > On Thu, Aug 27, 2020 at 7:05 PM Gerard Casas Saez > > <gcasass...@twitter.com.invalid> wrote: > > > > > Okay, then if noone objects before Monday 8AM MST, I'll assume lazy > > > consensus and bring the document text into the main CWiki page for AIP. > > > > > > Gerard Casas Saez > > > Twitter | Cortex | @casassaez <http://twitter.com/casassaez> > > > > > > > > > On Thu, Aug 27, 2020 at 2:21 AM Jarek Potiuk <jarek.pot...@polidea.com > > > > > wrote: > > > > > > > LGTM > > > > > > > > On Thu, Aug 27, 2020 at 3:09 AM Gerard Casas Saez > > > > <gcasass...@twitter.com.invalid> wrote: > > > > > > > > > > Did a new pass to define what changes may be considered worth an > AIP. > > > > > > > > > > Regarding GitHub issues, after seeing SIP-0 I quite liked it. Seems > > way > > > > > more lightweight and reduces issue of having to ask for permissions > > for > > > > it > > > > > to be accepted. I do think it may be out of scope for this > > conversation > > > > to > > > > > migrate, but I would be a +1 on that vote. > > > > > > > > > > Gerard Casas Saez > > > > > Twitter | Cortex | @casassaez <http://twitter.com/casassaez> > > > > > > > > > > > > > > > On Tue, Aug 25, 2020 at 12:20 PM Jarek Potiuk < > > > jarek.pot...@polidea.com> > > > > > wrote: > > > > > > > > > > > I also prefer Github Issues than either CWiki or PR. > > > > > > > > > > > > I think there is little contention and usually, one person at a > > time > > > > leads > > > > > > AIP description and updates the doc but others can still edit it > > > > directly > > > > > > for minor changes. Github issues has a history of changes (the > ... > > at > > > > the > > > > > > top) and this is enough to make sure the history of changes is > > > > trackable > > > > > > IMHO. > > > > > > > > > > > > The only thing that I like better about CWiki is draw.io > > > > integration. I > > > > > > agree with Max (the other thread) that pictures are often worth > > 1000 > > > > words > > > > > > and having them embedded is a big plus. But you can link to > > draw.io > > > > and > > > > > > embed images - a little involved but workable for discussion. So > I > > > > could do > > > > > > either CWilki or Github Issues. > > > > > > > > > > > > J > > > > > > > > > > > > On Tue, Aug 25, 2020 at 6:33 PM Kaxil Naik <kaxiln...@gmail.com> > > > > wrote: > > > > > > > > > > > > > I am actually against the idea of having to PRs for editing > > AIPs. I > > > > think > > > > > > > the current separation works better. > > > > > > > > > > > > > > Confluence is fine for me as it is because of its integrations. > > > Don't > > > > > > have > > > > > > > anything against Github issue but definitely not worth having > it > > in > > > > > > Airflow > > > > > > > docs which needs to be PR'd > > > > > > > > > > > > > > On Tue, Aug 25, 2020 at 4:47 PM Gerard Casas Saez > > > > > > > <gcasass...@twitter.com.invalid> wrote: > > > > > > > > > > > > > > > In that direction, I would prefer having markdown docs and > PRs > > > > instead > > > > > > > > actually, similarly to how github.com/tensorflow/community > > does > > > > it. > > > > > > > > Thoughts? > > > > > > > > > > > > > > > > Will do a second pass to the later today :D > > > > > > > > > > > > > > > > Gerard Casas Saez > > > > > > > > Twitter | Cortex | @casassaez <http://twitter.com/casassaez> > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Aug 25, 2020 at 6:57 AM Ry Walker <r...@rywalker.com> > > > wrote: > > > > > > > > > > > > > > > > > Yeah I’d like for AIPs to be GitHub issues, more likely to > > get > > > > better > > > > > > > > > attention since we’re all in GitHub way more than cwiki. > > > > > > > > > > > > > > > > > > On Tue, Aug 25, 2020 at 4:01 AM Jarek Potiuk < > > > > > > jarek.pot...@polidea.com > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > Looks great Gerard! I think maybe one last pass with > > > Superset's > > > > > > docs. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Side comment. I really like that SIP's of superset end up > > as > > > > Github > > > > > > > > > Issues. > > > > > > > > > > > > > > > > > > > > They even have template for it. Maybe we should do the > > same? > > > > Seems > > > > > > > that > > > > > > > > > we > > > > > > > > > > > > > > > > > > > > are gravitating towards "Github heads-on" recently - > > Github > > > > > > Actions, > > > > > > > > > > > > > > > > > > > > Github Registry, Github Issues, GitHub Discussions .... I > > > also > > > > > > > applied > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > Github CodeSpaces access . > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > J. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Aug 25, 2020 at 2:27 AM Gerard Casas Saez > > > > > > > > > > > > > > > > > > > > <gcasass...@twitter.com.invalid> wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks Max for sharing. I will take a look and check > what > > > we > > > > > > should > > > > > > > > > reuse > > > > > > > > > > > > > > > > > > > > > from there as well. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I also took a pass to the comments and edited + > accepted > > > > > > > suggestions. > > > > > > > > > Let > > > > > > > > > > > > > > > > > > > > > me know: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://docs.google.com/document/d/1hJDodc1xl4ixswPzAV4IAQu74sLnOOKjqecPgthLta4/edit > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Will try to read the document sent by Max tomorrow and > > add > > > > > > details > > > > > > > > > based > > > > > > > > > > on > > > > > > > > > > > > > > > > > > > > > that as needed. If someone wants to do that before > > tomorrow > > > > > > > > afternoon, > > > > > > > > > > > > > > > > > > > > > please let me know! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Gerard Casas Saez > > > > > > > > > > > > > > > > > > > > > Twitter | Cortex | @casassaez < > > > http://twitter.com/casassaez> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Mon, Aug 24, 2020 at 3:53 PM Maxime Beauchemin < > > > > > > > > > > > > > > > > > > > > > maximebeauche...@gmail.com> wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > For reference, here's SIP-0 (Superset Improvement > > > Proposal > > > > 0) > > > > > > > which > > > > > > > > > > > > > > > > > > > > > defines > > > > > > > > > > > > > > > > > > > > > > the SIP process for Apache Superset: > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-superset/issues/5602 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We also have a GitHub issue template for it: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-superset/blob/master/.github/ISSUE_TEMPLATE/sip.md > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > and this kanban project to track the status of each > > SIP: > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-superset/projects/7 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Max > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Sun, Aug 23, 2020 at 3:38 AM Tomasz Urbaszek < > > > > > > > > > turbas...@apache.org> > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks Gerard for this proposal, it looks good! I > > added > > > > few > > > > > > > > > > > > > > > > > > > > > suggestions. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Bests, > > > > > > > > > > > > > > > > > > > > > > > Tomek > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Aug 21, 2020 at 10:49 PM Kaxil Naik < > > > > > > > kaxiln...@gmail.com > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks Gerard, added my suggestions. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Aug 21, 2020 at 7:17 PM Jarek Potiuk < > > > > > > > > > > > > > > > > > > > > > jarek.pot...@polidea.com > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > It's super cool indeed! I already made my > > > > suggestions and > > > > > > > > > > > > > > > > > > > > > comments. I > > > > > > > > > > > > > > > > > > > > > > > > think > > > > > > > > > > > > > > > > > > > > > > > > > there is no need to do vote - lazy consensus > > should > > > > be > > > > > > > > applied > > > > > > > > > in > > > > > > > > > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > > case > > > > > > > > > > > > > > > > > > > > > > > > > I think when you think is ready :). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > J. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Aug 21, 2020 at 8:12 PM Gerard Casas > Saez > > > > > > > > > > > > > > > > > > > > > > > > > <gcasass...@twitter.com.invalid> wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I do have access, I will add it there once > > > there's > > > > > > > > consensus > > > > > > > > > > (not > > > > > > > > > > > > > > > > > > > > > > > sure > > > > > > > > > > > > > > > > > > > > > > > > if > > > > > > > > > > > > > > > > > > > > > > > > > > we should submit this to vote). > > > > > > > > > > > > > > > > > > > > > > > > > > Gerard Casas Saez > > > > > > > > > > > > > > > > > > > > > > > > > > Twitter | Cortex | @casassaez < > > > > > > > > http://twitter.com/casassaez> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Aug 21, 2020 at 12:09 PM Ry Walker < > > > > > > > > r...@rywalker.com> > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Proposal looks good to me Gerard - thanks > for > > > > putting > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > together! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -Ry > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Aug 21, 2020 at 1:41 PM Gerard > Casas > > > Saez > > > > > > > > > > > > > > > > > > > > > > > > > > > <gcasass...@twitter.com.invalid> wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi folks, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I realized that the AIP process was not > > super > > > > clear > > > > > > > at > > > > > > > > > > > > > > > > > > > > > times. I > > > > > > > > > > > > > > > > > > > > > > > was > > > > > > > > > > > > > > > > > > > > > > > > > > lucky > > > > > > > > > > > > > > > > > > > > > > > > > > > > when I proposed AIP-31 that Dan and Tomek > > > > guided me > > > > > > > > > through > > > > > > > > > > > > > > > > > > > > > > most > > > > > > > > > > > > > > > > > > > > > > > of > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > process themselves, but I realize this > may > > be > > > > > > > confusing > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > > new > > > > > > > > > > > > > > > > > > > > > > > > > members > > > > > > > > > > > > > > > > > > > > > > > > > > > of > > > > > > > > > > > > > > > > > > > > > > > > > > > > the community. For that reason and after > > > > discussing > > > > > > > > this > > > > > > > > > > with > > > > > > > > > > > > > > > > > > > > > > > Kamil > > > > > > > > > > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > > Jarek on Slack > > > > > > > > > > > > > > > > > > > > > > > > > > > > < > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://docs.google.com/document/d/1hJDodc1xl4ixswPzAV4IAQu74sLnOOKjqecPgthLta4/edit?usp=sharing > > > > > > > > > > > > > > > > > > > > > > > > > > > > >. > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm proposing to extend the AIP page to > > > > include a > > > > > > > > > > description > > > > > > > > > > > > > > > > > > > > > > of > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > lifecycle of the AIP. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I put a doc together with some of my > > > > understanding > > > > > > > as a > > > > > > > > > new > > > > > > > > > > > > > > > > > > > > > > > member > > > > > > > > > > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > how > > > > > > > > > > > > > > > > > > > > > > > > > > > > I saw the process for AIP-31. Suggestions > > are > > > > > > welcome > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > encouraged! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Document proposal: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://docs.google.com/document/d/1hJDodc1xl4ixswPzAV4IAQu74sLnOOKjqecPgthLta4/edit?usp=sharing > > > > > > > > > > > > > > > > > > > > > > > > > > > > < > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://docs.google.com/document/d/1hJDodc1xl4ixswPzAV4IAQu74sLnOOKjqecPgthLta4/edit?usp=sharing > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Let me know what you think about this :D > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Gerard Casas Saez > > > > > > > > > > > > > > > > > > > > > > > > > > > > Twitter | Cortex | @casassaez < > > > > > > > > > > http://twitter.com/casassaez> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Jarek Potiuk > > > > > > > > > > > > > > > > > > > > > > > > > Polidea <https://www.polidea.com/> | Principal > > > > Software > > > > > > > > > Engineer > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > M: +48 660 796 129 <+48660796129> > > > > > > > > > > > > > > > > > > > > > > > > > [image: Polidea] <https://www.polidea.com/> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Jarek Potiuk > > > > > > > > > > > > > > > > > > > > Polidea <https://www.polidea.com/> | Principal Software > > > > Engineer > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > M: +48 660 796 129 <+48660796129> > > > > > > > > > > > > > > > > > > > > [image: Polidea] <https://www.polidea.com/> > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > Sent from Gmail Mobile > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > > Jarek Potiuk > > > > > > Polidea <https://www.polidea.com/> | Principal Software Engineer > > > > > > > > > > > > M: +48 660 796 129 <+48660796129> > > > > > > [image: Polidea] <https://www.polidea.com/> > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > Jarek Potiuk > > > > Polidea | Principal Software Engineer > > > > > > > > M: +48 660 796 129 > > > > > > > > > > -- Jarek Potiuk Polidea <https://www.polidea.com/> | Principal Software Engineer M: +48 660 796 129 <+48660796129> [image: Polidea] <https://www.polidea.com/>
