airflow-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gerard Casas Saez <gcasass...@twitter.com.INVALID>
Subject Re: [Discuss] Improving AIP process documentation
Date Mon, 31 Aug 2020 16:51:57 GMT
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 <kaxilnaik@gmail.com> wrote:

> Yes go for it
>
> On Thu, Aug 27, 2020 at 7:05 PM Gerard Casas Saez
> <gcasassaez@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.Potiuk@polidea.com>
> > wrote:
> >
> > > LGTM
> > >
> > > On Thu, Aug 27, 2020 at 3:09 AM Gerard Casas Saez
> > > <gcasassaez@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.Potiuk@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 <kaxilnaik@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
> > > > > > <gcasassaez@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 <ry@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.Potiuk@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
> > > > > > > > >
> > > > > > > > > <gcasassaez@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
<
> > > > > > > > >
> > > > > > > > > > maximebeauchemin@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 <
> > > > > > > > turbaszek@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 <
> > > > > > kaxilnaik@gmail.com
> > > > > > > >
> > > > > > > > >
> > > > > > > > > > wrote:
> > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > >
> > > > > > > > > > > > > Thanks Gerard, added my suggestions.
> > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > >
> > > > > > > > > > > > > On Fri, Aug 21, 2020 at 7:17
PM Jarek Potiuk <
> > > > > > > > >
> > > > > > > > > > Jarek.Potiuk@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
> > > > > > > > >
> > > > > > > > > > > > > > <gcasassaez@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 <
> > > > > > > ry@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
> > > > > > > > >
> > > > > > > > > > > > > > > > <gcasassaez@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
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message