[COMMUNITY] Update maturity model #3743

pan3793 · 2022-11-01T14:36:47Z

Why are the changes needed?

This PR proposes to update the Kyuubi project maturity model to reflect the current project status.

How was this patch tested?

Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before make a pull request

codecov-commenter · 2022-11-01T15:54:56Z

Codecov Report

❗ No coverage uploaded for pull request base (master@bd35784). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3743   +/-   ##
=========================================
  Coverage          ?   52.64%           
  Complexity        ?       13           
=========================================
  Files             ?      494           
  Lines             ?    27788           
  Branches          ?     3835           
=========================================
  Hits              ?    14630           
  Misses            ?    11768           
  Partials          ?     1390

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

yaooqinn · 2022-11-02T02:03:51Z

I second to comments from our mentor, can you elaborate a bit more what&why can we update the model, especially add some solid evidence, not only says that update .... reflect the current project status. which is clear to everyone

pan3793 · 2022-11-02T05:47:37Z

MATURITY.md

+| CS20 | Decisions require a consensus among PMC members and are documented on the project's main communications channel. The PMC takes community opinions into account, but the PMC has the final word.                           | :white_check_mark: The project has been making important decisions on the mailing lists.                                                                                                                       |
+| CS30 | The project uses documented voting rules to build consensus when discussion is not sufficient.                                                                                                                            | :white_check_mark: The project uses the standard ASF voting rules.                                                                                                                                             |
+| CS40 | In Apache projects, vetoes are only valid for code commits. The person exercising the veto must justify it with a technical explanation, as per the Apache voting rules defined in CS30.                                  | :white_check_mark: The project has not used a veto at any point during incubation.                                                                                                                             |
+| CS50 | All "important" discussions happen asynchronously in written form on the project's main communications channel. Offline, face-to-face or private discussions that affect the project are also documented on that channel. | :white_check_mark: The project has been making important decisions on the project mailing lists. Minor decisions may occasionally happen during code reviews, which are also asynchronous and in written form. |


For example:

Major features discussion:

[DISCUSS][KPIP-4] Support to submit batch job

[KPIP-4] Support to submit batch job

[DISCUSS][KPIP-5] Introduce Kyuubi Web UI

[KPIP-5] Introduce Kyuubi Web UI

[VOTE][KPIP-5] Introduce Kyuubi Web UI

Deprecation discussion:

[DISCUSS] Spark 3.0 Support EOL Progress

pan3793 · 2022-11-02T05:54:24Z

MATURITY.md

-| IN20 | Contributors act as themselves, not as representatives of a corporation or organization. | :white_check_mark: The contributors act on their own initiative without representing a corporation or organization.                                                                                                                                                                                                |
+| ID   | Description                                                                              | Status                                                                                                                                                                                                                                                                                                            |
+|------|------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| IN10 | The project is independent from any corporate or organizational influence.               | :white_check_mark: The project team gathers people from different companies (AsiaInfo, bilibili, China Mobile, eBay, iQIYI, NetEase, T3, Zhihu ...). No company or organization has significantly more influence than any other. We can note a growth of the contributions coming from different contributors.    |


There are 12 PPMC members come from different companies, including AWS, eBay, Huawei, Sensors Data, NetEase, China Mobile, Ctrip, and T3
https://whimsy.apache.org/roster/ppmc/kyuubi

And the number of contributors is growing.
https://github.com/apache/incubator-kyuubi/graphs/contributors

pan3793 · 2022-11-02T06:00:37Z

MATURITY.md

+| CS10 | The project maintains a public list of its contributors who have decision power. The project's PMC (Project Management Committee) consists of those contributors.                                                         | :white_check_mark: Committers and PPMC members are listed on the website’s community page, see [newcommitter](https://kyuubi.apache.org/become_committer.html)                                                 |
+| CS20 | Decisions require a consensus among PMC members and are documented on the project's main communications channel. The PMC takes community opinions into account, but the PMC has the final word.                           | :white_check_mark: The project has been making important decisions on the mailing lists.                                                                                                                       |
+| CS30 | The project uses documented voting rules to build consensus when discussion is not sufficient.                                                                                                                            | :white_check_mark: The project uses the standard ASF voting rules.                                                                                                                                             |
+| CS40 | In Apache projects, vetoes are only valid for code commits. The person exercising the veto must justify it with a technical explanation, as per the Apache voting rules defined in CS30.                                  | :white_check_mark: The veto is used for code commits only, and each veto with a clear technical explanation.                                                                                                   |


For instance: #3486

pan3793 · 2022-11-02T06:04:17Z

@yaooqinn @WillemJiang I supplied examples to explain why I think the project reached such a standard.

ulysses-you · 2022-11-03T12:33:35Z

MATURITY.md

-| CS50 | All "important" discussions happen asynchronously in written form on the project's main communications channel. Offline, face-to-face or private discussions that affect the project are also documented on that channel. | :white_large_square: The project has been making important decisions on the project mailing lists. Minor decisions may occasionally happen during code reviews, which are also asynchronous and in written form. |
+| ID   | Description                                                                                                                                                                                                               | Status                                                                                                                                                                                                         |
+|------|---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| CS10 | The project maintains a public list of its contributors who have decision power. The project's PMC (Project Management Committee) consists of those contributors.                                                         | :white_check_mark: Committers and PPMC members are listed on the website’s community page, see [newcommitter](https://kyuubi.apache.org/become_committer.html)                                                 |


as we are here, add one more link [new_ppmc] https://kyuubi.apache.org/become_ppmc_member.html

yaooqinn · 2022-11-04T07:53:15Z

MATURITY.md

+| CS10 | The project maintains a public list of its contributors who have decision power. The project's PMC (Project Management Committee) consists of those contributors.                                                         | :white_check_mark: Committers and PPMC members are listed on the website’s community page, see [newcommitter](https://kyuubi.apache.org/become_committer.html), [newppmc](https://kyuubi.apache.org/become_pmc_member.html) |
+| CS20 | Decisions require a consensus among PMC members and are documented on the project's main communications channel. The PMC takes community opinions into account, but the PMC has the final word.                           | :white_check_mark: The project has been making important decisions on the mailing lists.                                                                                                                                    |
+| CS30 | The project uses documented voting rules to build consensus when discussion is not sufficient.                                                                                                                            | :white_check_mark: The project uses the standard ASF voting rules.                                                                                                                                                          |
+| CS40 | In Apache projects, vetoes are only valid for code commits. The person exercising the veto must justify it with a technical explanation, as per the Apache voting rules defined in CS30.                                  | :white_check_mark: The veto is used for code commits only, and each veto with a clear technical explanation.                                                                                                                |


Let's change this sentence from passive voice to active.

We've used vetoes with clear technical explanations for code commits only.

Thanks, updated

yaooqinn

LGTM, only one typo

pan3793 · 2022-11-06T16:45:31Z

Thanks all for your review and approval, the success of Kyuubi is inseparable from all of you! Merging to master.

github-actions bot added the kind:documentation Documentation is a feature! label Nov 1, 2022

pan3793 marked this pull request as ready for review November 1, 2022 14:37

pan3793 commented Nov 2, 2022

View reviewed changes

turboFei approved these changes Nov 3, 2022

View reviewed changes

cfmcgrady approved these changes Nov 3, 2022

View reviewed changes

cxzl25 approved these changes Nov 3, 2022

View reviewed changes

ulysses-you reviewed Nov 3, 2022

View reviewed changes

hddong approved these changes Nov 3, 2022

View reviewed changes

wForget approved these changes Nov 4, 2022

View reviewed changes

ulysses-you approved these changes Nov 4, 2022

View reviewed changes

WillemJiang approved these changes Nov 4, 2022

View reviewed changes

yaooqinn reviewed Nov 4, 2022

View reviewed changes

yaooqinn approved these changes Nov 4, 2022

View reviewed changes

SteNicholas approved these changes Nov 4, 2022

View reviewed changes

pan3793 self-assigned this Nov 5, 2022

pan3793 added this to the v1.7.0 milestone Nov 5, 2022

pan3793 added 4 commits November 5, 2022 08:14

[COMMUNITY] Update maturity model

a109bce

Update

f410d4d

update

221ccbb

polish

3ea746f

pan3793 force-pushed the maturity branch from bbec93f to 3ea746f Compare November 5, 2022 08:14

pan3793 added the kind:infra license, community building, project builds, asf infra related, etc. label Nov 6, 2022

pan3793 closed this in 77cfa16 Nov 6, 2022

pan3793 deleted the maturity branch November 13, 2022 17:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[COMMUNITY] Update maturity model #3743

[COMMUNITY] Update maturity model #3743

Uh oh!

pan3793 commented Nov 1, 2022

Uh oh!

codecov-commenter commented Nov 1, 2022 •

edited

Loading

Uh oh!

yaooqinn commented Nov 2, 2022

Uh oh!

pan3793 Nov 2, 2022

Uh oh!

pan3793 Nov 2, 2022 •

edited

Loading

Uh oh!

pan3793 Nov 2, 2022

Uh oh!

pan3793 commented Nov 2, 2022

Uh oh!

ulysses-you Nov 3, 2022

Uh oh!

yaooqinn Nov 4, 2022

Uh oh!

pan3793 Nov 4, 2022

Uh oh!

yaooqinn left a comment

Uh oh!

pan3793 commented Nov 6, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

[COMMUNITY] Update maturity model #3743

[COMMUNITY] Update maturity model #3743

Uh oh!

Conversation

pan3793 commented Nov 1, 2022

Why are the changes needed?

How was this patch tested?

Uh oh!

codecov-commenter commented Nov 1, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

yaooqinn commented Nov 2, 2022

Uh oh!

pan3793 Nov 2, 2022

Choose a reason for hiding this comment

Uh oh!

pan3793 Nov 2, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pan3793 Nov 2, 2022

Choose a reason for hiding this comment

Uh oh!

pan3793 commented Nov 2, 2022

Uh oh!

ulysses-you Nov 3, 2022

Choose a reason for hiding this comment

Uh oh!

yaooqinn Nov 4, 2022

Choose a reason for hiding this comment

Uh oh!

pan3793 Nov 4, 2022

Choose a reason for hiding this comment

Uh oh!

yaooqinn left a comment

Choose a reason for hiding this comment

Uh oh!

pan3793 commented Nov 6, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

codecov-commenter commented Nov 1, 2022 •

edited

Loading

pan3793 Nov 2, 2022 •

edited

Loading