yangjf2019 commented on a change in pull request #268:
URL: https://github.com/apache/flink-web/pull/268#discussion_r422998326
##########
File path: contributing/contribute-code.zh.md
##########
@@ -126,111 +126,111 @@ Apache Flink is maintained, improved, and extended by
code contributions of volu
<div class="col-sm-12">
<div class="panel panel-default">
<div class="panel-body">
- Note: <i>trivial</i> hot fixes such as typos or syntax errors can be
opened as a <code>[hotfix]</code> pull request, without a Jira ticket.
+ 注意：诸如拼写错误或语法错误之类的<i>简单</i>热修复可以在打开 pull request 时，使用 [hotfix] 标识，可以不用
Jira ticket。
</div>
</div>
</div>
</div>
-<a name="consensus"></a>
+<a name="达成共识"></a>
-### 1. Create Jira Ticket and Reach Consensus
+### 1. 创建 Jira 工单并达成共识。
-The first step for making a contribution to Apache Flink is to reach consensus
with the Flink community. This means agreeing on the scope and implementation
approach of a change.
+向 Apache Flink 做出贡献的第一步是与 Flink 社区达成共识，这意味着需要一起商定更改的范围和实现的方法。
-In most cases, the discussion should happen in [Flink's bug tracker:
Jira](https://issues.apache.org/jira/projects/FLINK/summary).
+在大多数情况下，讨论应该发生在 [Flink 的 Bug
跟踪器：Jira](https://issues.apache.org/jira/projects/FLINK/summary)中。
-The following types of changes require a `[DISCUSS]` thread on the
[email protected] Flink mailing list:
+以下类型的更改需要在 [email protected] Flink 邮件列表中标识 `[DISCUSS]`:
- - big changes (major new feature; big refactorings, involving multiple
components)
- - potentially controversial changes or issues
- - changes with very unclear approaches or multiple equal approaches
+ - 重大变化（主要新功能、大重构和涉及多个组件）
+ - 可能存在争议的变化或问题
+ - 采用非常不明确的方法或多种相同方法的变化
- Do not open a Jira ticket for these types of changes before the discussion
has come to a conclusion.
- Jira tickets based on a dev@ discussion need to link to that discussion and
should summarize the outcome.
+ 在讨论结束之前,不要为这些类型的更改打开 Jira 工单。
+ 基于 dev 邮件讨论的 Jira 工单需要链接到该讨论，并总结结果。
-**Requirements for a Jira ticket to get consensus:**
+**Jira 工单获得共识的要求：**
- - Formal requirements
- - The *Title* describes the problem concisely.
- - The *Description* gives all the details needed to understand the
problem or feature request.
- - The *Component* field is set: Many committers and contributors only
focus on certain subsystems of Flink. Setting the appropriate component is
important for getting their attention.
- - There is **agreement** that the ticket solves a valid problem, and that it
is a **good fit** for Flink.
- The Flink community considers the following aspects:
- - Does the contribution alter the behavior of features or components in a
way that it may break previous users’ programs and setups? If yes, there needs
to be a discussion and agreement that this change is desirable.
- - Does the contribution conceptually fit well into Flink? Is it too much
of a special case such that it makes things more complicated for the common
case, or bloats the abstractions / APIs?
- - Does the feature fit well into Flink’s architecture? Will it scale and
keep Flink flexible for the future, or will the feature restrict Flink in the
future?
- - Is the feature a significant new addition (rather than an improvement
to an existing part)? If yes, will the Flink community commit to maintaining
this feature?
- - Does this feature align well with Flink's roadmap and currently ongoing
efforts?
- - Does the feature produce added value for Flink users or developers? Or
does it introduce the risk of regression without adding relevant user or
developer benefit?
- - Could the contribution live in another repository, e.g., Apache Bahir
or another external repository?
- - Is this a contribution just for the sake of getting a commit in an open
source project (fixing typos, style changes merely for taste reasons)
- - There is **consensus** on how to solve the problem. This includes
considerations such as
- - API and data backwards compatibility and migration strategies
- - Testing strategies
- - Impact on Flink's build time
- - Dependencies and their licenses
+ - 正式要求
+ - 描述问题的 *Title* 要简明扼要。
+ - 在 *Description* 中要提供了解问题或功能请求所需的所有详细信息。
+ - 要设置 *Component* 字段：许多 committers 和贡献者，只专注于 Flink
的某些子系统。设置适当的组件标签对于引起他们的注意很重要。
+ - 社区*一致同意*使用 tickets 是有效解决问题的方法，而且这**非常适合** Flink。
+ Flink 社区考虑了以下几个方面：
+ - 这种贡献是否会改变特性或组件的性能，从而破坏以前的用户程序和设置？如果是，那么就需要讨论并达成一致意见，证明这种改变是可取的。
+ - 这个贡献在概念上是否适合 Flink ？这是一个特殊情况，是因为它使简单问题变复杂，还是使 abstractions 或者 APIs 变得臃肿？
+ - 该功能是否适合 Flink 的架构？它是否易扩展并保持 Flink 未来的灵活性，或者该功能将来会限制 Flink 吗？
+ - 该特性是一个重要的新增(而不是对现有部分的改进)吗？如果是，Flink 社区会承诺维护这个特性吗？
+ - 这个特性是否与 Flink 的路线图以及当前正在进行的工作内容一致？
+ - 该特性是否为 Flink 用户或开发人员带来了附加价值？或者它引入了回归的风险而没有给相关的用户或开发人员带来好处？
+ - 该贡献是否存在于另一个存储库中，例如 Apache Bahir 或另一个外部存储库？
+ - 这仅仅是为了在开源项目中获得提交而做出的贡献吗（仅仅是为了获得贡献而贡献，才去修复拼写错误、改变代码风格）?
+ - 在如何解决这个问题上已有**共识**，包括以下需要考虑的因素
+ - API 、数据向后兼容性和迁移策略
+ - 测试策略
+ - 对 Flink 构建时间的影响
+ - 依赖关系及其许可证
-If a change is identified as a large or controversial change in the discussion
on Jira, it might require a [Flink Improvement Proposal
(FLIP)](https://cwiki.apache.org/confluence/display/FLINK/Flink+Improvement+Proposals)
or a discussion on the [dev mailing list]( {{ site.base
}}/community.html#mailing-lists) to reach agreement and consensus.
+如果在 Jira 的讨论中发现改动是一个大的或有争议的变更，则可能需要有 [Flink 改动建议 ( FLIP
)](https://cwiki.apache.org/confluence/display/FLINK/Flink+Improvement+Proposals)
或在 [ dev 邮件列表]( {{ site.base }}/community.html#mailing-lists) 中讨论以达成一致意见。
-Contributors can expect to get a first reaction from a committer within a few
days after opening the ticket. If a ticket doesn't get any attention, we
recommend reaching out to the [developer mailing list]( {{ site.base
}}/community.html#mailing-lists). Note that the Flink community sometimes does
not have the capacity to accept all incoming contributions.
+贡献者可以在打开 ticket 后的几天内得到来自 committer 的第一回应。如果 ticket 没有得到任何关注，我们建议你联系 [dev
邮件列表]( {{ site.base }}/community.html#mailing-lists)。请注意，Flink
社区有时没有能力接受发来的所有贡献信息。
-Once all requirements for the ticket are met, a committer will assign somebody
to the *`Assignee`* field of the ticket to work on it.
-Only committers have the permission to assign somebody.
+一旦满足了 ticket 的所有要求，committer 就会将某人*`分配`*给 ticket 的受理人以进行处理。
+只有 committer 才有权限分配 ticket 给某人。
-**Pull requests belonging to unassigned Jira tickets will not be reviewed or
merged by the community**.
+**社区不会审查或合并属于未分配的 Jira ticket 的 pull request ！**
-<a name="implement"></a>
+<a name="实现"></a>
-### 2. Implement your change
+### 2. 实现你想改动的
-Once you've been assigned to a Jira issue, you may start to implement the
required changes.
+一旦你被分配到了 Jira issue，你就可以开始去实现你想改动的内容。
-Here are some further points to keep in mind while implementing:
+以下是在实现时要注意的一些要点：
-- [Set up a Flink development
environment](https://cwiki.apache.org/confluence/display/FLINK/Setting+up+a+Flink+development+environment)
-- Follow the [Code Style and Quality Guide]({{ site.base
}}/contributing/code-style-and-quality.html) of Flink
-- Take any discussions and requirements from the Jira issue or design document
into account.
-- Do not mix unrelated issues into one contribution.
+- [设置 Flink
的开发环境](https://cwiki.apache.org/confluence/display/FLINK/Setting+up+a+Flink+development+environment)
+- 遵循 Flink 的 [代码风格和质量指南]({{ site.base
}}/contributing/code-style-and-quality.html)
+- 接受来自 Jira issue 或设计文档中的任何讨论和要求。
+- 不要将不相关的问题混合到一个贡献中。
-<a name="review"></a>
+<a name="审查"></a>
-### 3. Open a Pull Request
+### 3. 打开一个 Pull Request
-Considerations before opening a pull request:
+在打开 pull request 之前的注意事项：
- - Make sure that **`mvn clean verify`** is passing on your changes to ensure
that all checks pass, the code builds and that all tests pass.
- - Execute the [End to End tests of
Flink](https://github.com/apache/flink/tree/master/flink-end-to-end-tests#running-tests).
- - Make sure no unrelated or unnecessary reformatting changes are included.
- - Make sure your commit history adheres to the requirements.
- - Make sure your change has been rebased to the latest commits in your base
branch.
- - Make sure the pull request refers to the respective Jira, and that each
Jira issue is assigned to exactly one pull request (in case of multiple pull
requests for one Jira; resolve that situation first)
+ - 确保 **`mvn clean verify`** 验证了你的更改，以确保所有检查都通过，代码构建并且所有测试都通过。
+ - 执行 [Flink
的端到端测试](https://github.com/apache/flink/tree/master/flink-end-to-end-tests#running-tests)。
+ - 确保不包含任何不相关或不必要的格式化更改。
+ - 确保你的提交历史符合要求。
+ - 确保更改是重新基于你的基本分支中的最新提交。
Review comment:
“确保你的改动是基于当前分支提交的 Pull request。”
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
