| |
| <!DOCTYPE html> |
| <html lang="zh" dir=ZgotmplZ> |
| |
| <head> |
| |
| |
| |
| <link rel="stylesheet" href="/bootstrap/css/bootstrap.min.css"> |
| <script src="/bootstrap/js/bootstrap.bundle.min.js"></script> |
| <link rel="stylesheet" type="text/css" href="/font-awesome/css/font-awesome.min.css"> |
| <script src="/js/anchor.min.js"></script> |
| <script src="/js/flink.js"></script> |
| <link rel="canonical" href="https://flink.apache.org/zh/how-to-contribute/reviewing-prs/"> |
| |
| <meta charset="UTF-8"> |
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> |
| <meta name="description" content="如何审核 Pull Request # 本指南适用于希望帮助审核代码的所有提交者和贡献者。感谢你的努力 - 良好的审核是开源项目中最重要也是最关键的部分之一。本文旨在协助社区开展代码审核工作，以达到下列目的： |
| 让贡献者拥有良好的贡献体验。 将审核过程结构化，以涵盖所有需要检查的重要方面。 保持 Flink 代码的高质量。 避免贡献者和审核者花费大量时间完善代码却最终被拒绝提交的情况。 审核清单 # 每次审核都需要检查以下六个方面。 我们建议按照以下顺序进行检查，以避免在还没有就是否添加某项功能或需要改动达成共识之前或没有满足一些正式条件前，就花费时间进行详细的代码质量审核。 |
| 1. 贡献的描述是否清晰？ # 检查贡献是否有充分的描述以方便审核，不重要的更改和修复不需要很长的描述。如果实现方案完全是按照之前在 Jira 或 dev 邮件列表上讨论结论进行的话，只需要一个对讨论的简短的引用即可。 如果实现方案与之前达成一致的方案不同的话，关于实现的详细描述是需要的，以便 review 贡献时更深入地讨论。 |
| 任何改变功能或行为的 pull request 都需要描述这些改变的重点, 以便知道审核什么内容(并且不必钻研代码来了解更改的作用)。 |
| 如果在不查看代码的情况下能回答以下问题2、3、4，则该贡献得到了很好的描述。 |
| 2. 是否一致认为这一变更或者功能应该进入 Flink？ # 这个问题要直接在关联的 Jira issue 中回答。对于在达成一致前创建的 pull request 来说，需要先在 Jira 中寻求一致的意见。 |
| 对于 [hotfix] 类型的的 pull request，可以在 pull request 中寻求意见一致。 |
| 3. 贡献是否需要一些特定的 committer 的关注，这些 committer 有时间投入吗？ # 一些更改需要特定的 committer 的注意和批准。例如，对性能非常敏感或对分布式协调和容错有关键影响的部件中的更改，这需要一个对相应组件非常熟悉的 committer 的审核。 |
| 根据经验，当 pull request 描述中对模板里问题 “Does this pull request potentially affect one of the following parts” 的回答为 “yes” 时，需要特别注意。"> |
| <meta name="theme-color" content="#FFFFFF"><meta property="og:title" content="审核 Pull Request" /> |
| <meta property="og:description" content="如何审核 Pull Request # 本指南适用于希望帮助审核代码的所有提交者和贡献者。感谢你的努力 - 良好的审核是开源项目中最重要也是最关键的部分之一。本文旨在协助社区开展代码审核工作，以达到下列目的： |
| 让贡献者拥有良好的贡献体验。 将审核过程结构化，以涵盖所有需要检查的重要方面。 保持 Flink 代码的高质量。 避免贡献者和审核者花费大量时间完善代码却最终被拒绝提交的情况。 审核清单 # 每次审核都需要检查以下六个方面。 我们建议按照以下顺序进行检查，以避免在还没有就是否添加某项功能或需要改动达成共识之前或没有满足一些正式条件前，就花费时间进行详细的代码质量审核。 |
| 1. 贡献的描述是否清晰？ # 检查贡献是否有充分的描述以方便审核，不重要的更改和修复不需要很长的描述。如果实现方案完全是按照之前在 Jira 或 dev 邮件列表上讨论结论进行的话，只需要一个对讨论的简短的引用即可。 如果实现方案与之前达成一致的方案不同的话，关于实现的详细描述是需要的，以便 review 贡献时更深入地讨论。 |
| 任何改变功能或行为的 pull request 都需要描述这些改变的重点, 以便知道审核什么内容(并且不必钻研代码来了解更改的作用)。 |
| 如果在不查看代码的情况下能回答以下问题2、3、4，则该贡献得到了很好的描述。 |
| 2. 是否一致认为这一变更或者功能应该进入 Flink？ # 这个问题要直接在关联的 Jira issue 中回答。对于在达成一致前创建的 pull request 来说，需要先在 Jira 中寻求一致的意见。 |
| 对于 [hotfix] 类型的的 pull request，可以在 pull request 中寻求意见一致。 |
| 3. 贡献是否需要一些特定的 committer 的关注，这些 committer 有时间投入吗？ # 一些更改需要特定的 committer 的注意和批准。例如，对性能非常敏感或对分布式协调和容错有关键影响的部件中的更改，这需要一个对相应组件非常熟悉的 committer 的审核。 |
| 根据经验，当 pull request 描述中对模板里问题 “Does this pull request potentially affect one of the following parts” 的回答为 “yes” 时，需要特别注意。" /> |
| <meta property="og:type" content="article" /> |
| <meta property="og:url" content="https://flink.apache.org/zh/how-to-contribute/reviewing-prs/" /><meta property="article:section" content="how-to-contribute" /> |
| |
| |
| <title>审核 Pull Request | Apache Flink</title> |
| <link rel="manifest" href="/manifest.json"> |
| <link rel="icon" href="/favicon.png" type="image/x-icon"> |
| <link rel="alternate" hreflang="en" href="https://flink.apache.org/how-to-contribute/reviewing-prs/" title="Review Pull Requests"> |
| |
| <link rel="stylesheet" href="/book.min.22eceb4d17baa9cdc0f57345edd6f215a40474022dfee39b63befb5fb3c596b5.css" integrity="sha256-IuzrTRe6qc3A9XNF7dbyFaQEdAIt/uObY777X7PFlrU="> |
| <script defer src="/zh.search.min.4fed8368d7410bf2cb8853d136d796128533433358adcd85133b7d8014ce76ef.js" integrity="sha256-T+2DaNdBC/LLiFPRNteWEoUzQzNYrc2FEzt9gBTOdu8="></script> |
| <!-- |
| Made with Book Theme |
| https://github.com/alex-shpak/hugo-book |
| --> |
| |
| <meta name="generator" content="Hugo 0.124.1"> |
| |
| |
| <script> |
| var _paq = window._paq = window._paq || []; |
| |
| |
| _paq.push(['disableCookies']); |
| |
| _paq.push(["setDomains", ["*.flink.apache.org","*.nightlies.apache.org/flink"]]); |
| _paq.push(['trackPageView']); |
| _paq.push(['enableLinkTracking']); |
| (function() { |
| var u="//analytics.apache.org/"; |
| _paq.push(['setTrackerUrl', u+'matomo.php']); |
| _paq.push(['setSiteId', '1']); |
| var d=document, g=d.createElement('script'), s=d.getElementsByTagName('script')[0]; |
| g.async=true; g.src=u+'matomo.js'; s.parentNode.insertBefore(g,s); |
| })(); |
| </script> |
| |
| </head> |
| |
| <body dir=ZgotmplZ> |
| |
| |
| |
| <header> |
| <nav class="navbar navbar-expand-xl"> |
| <div class="container-fluid"> |
| <a class="navbar-brand" href="/zh/"> |
| <img src="/img/logo/png/100/flink_squirrel_100_color.png" alt="Apache Flink" height="47" width="47" class="d-inline-block align-text-middle"> |
| <span>Apache Flink</span> |
| </a> |
| <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent" aria-expanded="false" aria-label="Toggle navigation"> |
| <i class="fa fa-bars navbar-toggler-icon"></i> |
| </button> |
| <div class="collapse navbar-collapse" id="navbarSupportedContent"> |
| <ul class="navbar-nav"> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| <li class="nav-item dropdown"> |
| <a class="nav-link dropdown-toggle" href="#" role="button" data-bs-toggle="dropdown" aria-expanded="false">About</a> |
| <ul class="dropdown-menu"> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/flink-architecture/">架构</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/flink-applications/">应用</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/flink-operations/">运维</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/use-cases/">应用场景</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/powered-by/">Flink 用户</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/roadmap/">开发计划</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/community/">社区 & 项目信息</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/security/">Security</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/what-is-flink/special-thanks/">特殊致谢</a> |
| |
| |
| </li> |
| |
| </ul> |
| </li> |
| |
| |
| |
| |
| |
| <li class="nav-item dropdown"> |
| <a class="nav-link dropdown-toggle" href="#" role="button" data-bs-toggle="dropdown" aria-expanded="false">教程</a> |
| <ul class="dropdown-menu"> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-docs-stable/docs/try-flink/local_installation/">With Flink<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-kubernetes-operator-docs-stable/docs/try-flink-kubernetes-operator/quick-start/">With Flink Kubernetes Operator<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-cdc-docs-stable/docs/get-started/introduction/">With Flink CDC<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-ml-docs-stable/docs/try-flink-ml/quick-start/">With Flink ML<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-statefun-docs-stable/getting-started/project-setup.html">With Flink Stateful Functions<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-docs-stable/docs/learn-flink/overview/">Training Course<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| </ul> |
| </li> |
| |
| |
| |
| |
| |
| <li class="nav-item dropdown"> |
| <a class="nav-link dropdown-toggle" href="#" role="button" data-bs-toggle="dropdown" aria-expanded="false">Documentation</a> |
| <ul class="dropdown-menu"> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-docs-stable/">Flink 1.19 (stable)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-docs-master/">Flink Master (snapshot)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-kubernetes-operator-docs-stable/">Kubernetes Operator 1.8 (latest)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-kubernetes-operator-docs-main">Kubernetes Operator Main (snapshot)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-cdc-docs-stable">CDC 3.0 (stable)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-cdc-docs-master">CDC Master (snapshot)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-ml-docs-stable/">ML 2.3 (stable)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-ml-docs-master">ML Master (snapshot)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-statefun-docs-stable/">Stateful Functions 3.3 (stable)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="https://nightlies.apache.org/flink/flink-statefun-docs-master">Stateful Functions Master (snapshot)<i class="link fa fa-external-link title" aria-hidden="true"></i> |
| </a> |
| |
| |
| </li> |
| |
| </ul> |
| </li> |
| |
| |
| |
| |
| |
| <li class="nav-item dropdown"> |
| <a class="nav-link dropdown-toggle" href="#" role="button" data-bs-toggle="dropdown" aria-expanded="false">How to Contribute</a> |
| <ul class="dropdown-menu"> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/overview/">如何参与贡献</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/contribute-code/">贡献代码</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/reviewing-prs/">审核 Pull Request</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/code-style-and-quality-preamble/">代码样式与质量指南</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/contribute-documentation/">贡献文档</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/documentation-style-guide/">文档样式指南</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/improve-website/">贡献网站</a> |
| |
| |
| </li> |
| |
| <li> |
| |
| |
| <a class="dropdown-item" href="/zh/how-to-contribute/getting-help/">获取帮助</a> |
| |
| |
| </li> |
| |
| </ul> |
| </li> |
| |
| |
| |
| |
| |
| |
| |
| |
| <li class="nav-item"> |
| |
| |
| <a class="nav-link" href="/zh/downloads/">Downloads</a> |
| |
| |
| </li> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| </ul> |
| <div class="book-search"> |
| <div class="book-search-spinner hidden"> |
| <i class="fa fa-refresh fa-spin"></i> |
| </div> |
| <form class="search-bar d-flex" onsubmit="return false;"su> |
| <input type="text" id="book-search-input" placeholder="搜索" aria-label="搜索" maxlength="64" data-hotkeys="s/"> |
| <i class="fa fa-search search"></i> |
| <i class="fa fa-circle-o-notch fa-spin spinner"></i> |
| </form> |
| <div class="book-search-spinner hidden"></div> |
| <ul id="book-search-results"></ul> |
| </div> |
| </div> |
| </div> |
| </nav> |
| <div class="navbar-clearfix"></div> |
| </header> |
| |
| |
| <main class="flex"> |
| <section class="container book-page"> |
| |
| <article class="markdown"><h1 id="如何审核-pull-request"> |
| 如何审核 Pull Request |
| <a class="anchor" href="#%e5%a6%82%e4%bd%95%e5%ae%a1%e6%a0%b8-pull-request">#</a> |
| </h1> |
| <p>本指南适用于希望帮助审核代码的所有提交者和贡献者。感谢你的努力 - 良好的审核是开源项目中最重要也是最关键的部分之一。本文旨在协助社区开展代码审核工作，以达到下列目的：</p> |
| <ul> |
| <li>让贡献者拥有良好的贡献体验。</li> |
| <li>将审核过程结构化，以涵盖所有需要检查的重要方面。</li> |
| <li>保持 Flink 代码的高质量。</li> |
| <li>避免贡献者和审核者花费大量时间完善代码却最终被拒绝提交的情况。</li> |
| </ul> |
| <h2 id="审核清单"> |
| 审核清单 |
| <a class="anchor" href="#%e5%ae%a1%e6%a0%b8%e6%b8%85%e5%8d%95">#</a> |
| </h2> |
| <p>每次审核都需要检查以下六个方面。 <strong>我们建议按照以下顺序进行检查，以避免在还没有就是否添加某项功能或需要改动达成共识之前或没有满足一些正式条件前，就花费时间进行详细的代码质量审核。</strong></p> |
| <h3 id="1-贡献的描述是否清晰"> |
| 1. 贡献的描述是否清晰？ |
| <a class="anchor" href="#1-%e8%b4%a1%e7%8c%ae%e7%9a%84%e6%8f%8f%e8%bf%b0%e6%98%af%e5%90%a6%e6%b8%85%e6%99%b0">#</a> |
| </h3> |
| <p>检查贡献是否有充分的描述以方便审核，不重要的更改和修复不需要很长的描述。如果实现方案完全是<a href="/zh/how-to-contribute/contribute-code/#consensus">按照之前在 Jira 或 dev 邮件列表上讨论结论</a>进行的话，只需要一个对讨论的简短的引用即可。 如果实现方案与之前达成一致的方案不同的话，关于实现的详细描述是需要的，以便 review 贡献时更深入地讨论。</p> |
| <p>任何改变功能或行为的 pull request 都需要描述这些改变的重点, 以便知道审核什么内容(并且不必钻研代码来了解更改的作用)。</p> |
| <p><strong>如果在不查看代码的情况下能回答以下问题2、3、4，则该贡献得到了很好的描述。</strong></p> |
| <hr> |
| <h3 id="2-是否一致认为这一变更或者功能应该进入-flink"> |
| 2. 是否一致认为这一变更或者功能应该进入 Flink？ |
| <a class="anchor" href="#2-%e6%98%af%e5%90%a6%e4%b8%80%e8%87%b4%e8%ae%a4%e4%b8%ba%e8%bf%99%e4%b8%80%e5%8f%98%e6%9b%b4%e6%88%96%e8%80%85%e5%8a%9f%e8%83%bd%e5%ba%94%e8%af%a5%e8%bf%9b%e5%85%a5-flink">#</a> |
| </h3> |
| <p>这个问题要直接在关联的 Jira issue 中回答。对于在达成一致前创建的 pull request 来说，需要<a href="/zh/how-to-contribute/contribute-code/">先在 Jira 中寻求一致的意见</a>。</p> |
| <p>对于 <code>[hotfix]</code> 类型的的 pull request，可以在 pull request 中寻求意见一致。</p> |
| <hr> |
| <h3 id="3-贡献是否需要一些特定的-committer-的关注这些-committer-有时间投入吗"> |
| 3. 贡献是否需要一些特定的 committer 的关注，这些 committer 有时间投入吗？ |
| <a class="anchor" href="#3-%e8%b4%a1%e7%8c%ae%e6%98%af%e5%90%a6%e9%9c%80%e8%a6%81%e4%b8%80%e4%ba%9b%e7%89%b9%e5%ae%9a%e7%9a%84-committer-%e7%9a%84%e5%85%b3%e6%b3%a8%e8%bf%99%e4%ba%9b-committer-%e6%9c%89%e6%97%b6%e9%97%b4%e6%8a%95%e5%85%a5%e5%90%97">#</a> |
| </h3> |
| <p>一些更改需要特定的 committer 的注意和批准。例如，对性能非常敏感或对分布式协调和容错有关键影响的部件中的更改，这需要一个对相应组件非常熟悉的 committer 的审核。</p> |
| <p>根据经验，当 pull request 描述中对模板里问题 “Does this pull request potentially affect one of the following parts” 的回答为 “yes” 时，需要特别注意。</p> |
| <p>这个问题可以参考如下回答</p> |
| <ul> |
| <li><em>Does not need specific attention</em></li> |
| <li><em>Needs specific attention for X (X 可以是例如 checkpointing、jobmanager 等)</em></li> |
| <li><em>Has specific attention for X by @committerA, @contributorB</em></li> |
| </ul> |
| <p><strong>如果 pull request 需要特别关注，则其中一个标记的 committers 或 contributors 应该给出最终批准。</strong></p> |
| <hr> |
| <h3 id="4-实现方案是否遵循了商定的整体方案架构"> |
| 4. 实现方案是否遵循了商定的整体方案/架构？ |
| <a class="anchor" href="#4-%e5%ae%9e%e7%8e%b0%e6%96%b9%e6%a1%88%e6%98%af%e5%90%a6%e9%81%b5%e5%be%aa%e4%ba%86%e5%95%86%e5%ae%9a%e7%9a%84%e6%95%b4%e4%bd%93%e6%96%b9%e6%a1%88%e6%9e%b6%e6%9e%84">#</a> |
| </h3> |
| <p>在这一步中，我们会检查一个贡献的实现是否遵循了在 Jira 或邮件列表中商定的方案。 这个问题应该尽可能地从 pull request 描述（或链接的 Jira ）中得到回答。</p> |
| <p>我们建议你在深入了解更改的各个部分进行评论之前先检查这一点。</p> |
| <hr> |
| <h3 id="5-整体代码质量是否良好是否符合我们希望在-flink-中维护的标准"> |
| 5. 整体代码质量是否良好，是否符合我们希望在 Flink 中维护的标准？ |
| <a class="anchor" href="#5-%e6%95%b4%e4%bd%93%e4%bb%a3%e7%a0%81%e8%b4%a8%e9%87%8f%e6%98%af%e5%90%a6%e8%89%af%e5%a5%bd%e6%98%af%e5%90%a6%e7%ac%a6%e5%90%88%e6%88%91%e4%bb%ac%e5%b8%8c%e6%9c%9b%e5%9c%a8-flink-%e4%b8%ad%e7%bb%b4%e6%8a%a4%e7%9a%84%e6%a0%87%e5%87%86">#</a> |
| </h3> |
| <p>这是对实际变更的详细代码审核，包括：</p> |
| <ul> |
| <li>变更是否按照 Jira issue 或 pull request 说明中的描述进行？</li> |
| <li>代码是否遵循正确的软件工程实践？代码是否正确、健壮、可维护、可测试？</li> |
| <li>在更改性能敏感部分时，是否对性能进行了优化？</li> |
| <li>测试是否覆盖了全部改动？</li> |
| <li>测试执行速度是否够快？（是否仅在必要时才使用重量级集成测试？）</li> |
| <li>代码格式是否遵循 Flink 的 checkstyle 模式？</li> |
| <li>代码是否避免引入额外的编译器警告？</li> |
| <li>如果依赖更新了，NOTICE 文件是否也更新了？</li> |
| </ul> |
| <p>可以在 <a href="/zh/how-to-contribute/code-style-and-quality-preamble/">Flink代码样式和质量指南</a> 中找到编码的规范和指南。</p> |
| <hr> |
| <h3 id="6-英文和中文文档是否都更新了"> |
| 6. 英文和中文文档是否都更新了？ |
| <a class="anchor" href="#6-%e8%8b%b1%e6%96%87%e5%92%8c%e4%b8%ad%e6%96%87%e6%96%87%e6%a1%a3%e6%98%af%e5%90%a6%e9%83%bd%e6%9b%b4%e6%96%b0%e4%ba%86">#</a> |
| </h3> |
| <p>如果这个 pull request 引入了一个新功能，该功能应该被文档化。Flink 社区正在同时维护英文和中文文档。所以如果你想要更新或扩展文档，英文和中文文档都需要更新。如果你不熟悉中文，请创建一个用于中文文档翻译的 JIRA 并附上 <code>chinese-translation</code> 的组件名，并与当前 JIRA 关联起来。如果你熟悉中文，我们鼓励在一个 pull request 中同时更新两边的文档。</p> |
| <p>阅读<a href="/zh/how-to-contribute/contribute-documentation/">如何贡献文档</a>了解更多。</p> |
| </article> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| <div class="edit-this-page"> |
| <p> |
| <a href="https://cwiki.apache.org/confluence/display/FLINK/Flink+Translation+Specifications">Want to contribute translation?</a> |
| </p> |
| <p> |
| <a href="//github.com/apache/flink-web/edit/asf-site/docs/content.zh/how-to-contribute/reviewing-prs.md"> |
| Edit This Page<i class="fa fa-edit fa-fw"></i> |
| </a> |
| </p> |
| </div> |
| |
| </section> |
| |
| <aside class="book-toc"> |
| |
| |
| |
| <nav id="TableOfContents"><h3>On This Page <a href="javascript:void(0)" class="toc" onclick="collapseToc()"><i class="fa fa-times" aria-hidden="true"></i></a></h3> |
| <ul> |
| <li><a href="#如何审核-pull-request">如何审核 Pull Request</a> |
| <ul> |
| <li><a href="#审核清单">审核清单</a> |
| <ul> |
| <li><a href="#1-贡献的描述是否清晰">1. 贡献的描述是否清晰？</a></li> |
| <li><a href="#2-是否一致认为这一变更或者功能应该进入-flink">2. 是否一致认为这一变更或者功能应该进入 Flink？</a></li> |
| <li><a href="#3-贡献是否需要一些特定的-committer-的关注这些-committer-有时间投入吗">3. 贡献是否需要一些特定的 committer 的关注，这些 committer 有时间投入吗？</a></li> |
| <li><a href="#4-实现方案是否遵循了商定的整体方案架构">4. 实现方案是否遵循了商定的整体方案/架构？</a></li> |
| <li><a href="#5-整体代码质量是否良好是否符合我们希望在-flink-中维护的标准">5. 整体代码质量是否良好，是否符合我们希望在 Flink 中维护的标准？</a></li> |
| <li><a href="#6-英文和中文文档是否都更新了">6. 英文和中文文档是否都更新了？</a></li> |
| </ul> |
| </li> |
| </ul> |
| </li> |
| </ul> |
| </nav> |
| |
| |
| </aside> |
| <aside class="expand-toc hidden"> |
| <a class="toc" onclick="expandToc()" href="javascript:void(0)"> |
| <i class="fa fa-bars" aria-hidden="true"></i> |
| </a> |
| </aside> |
| |
| </main> |
| |
| <footer> |
| |
| |
| |
| <div class="separator"></div> |
| <div class="panels"> |
| <div class="wrapper"> |
| <div class="panel"> |
| <ul> |
| <li> |
| <a href="https://flink-packages.org/">flink-packages.org</a> |
| </li> |
| <li> |
| <a href="https://www.apache.org/">Apache Software Foundation</a> |
| </li> |
| <li> |
| <a href="https://www.apache.org/licenses/">License</a> |
| </li> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| <li> |
| <a href="/how-to-contribute/reviewing-prs/"> |
| <i class="fa fa-globe" aria-hidden="true"></i> English |
| </a> |
| </li> |
| |
| |
| |
| |
| |
| |
| |
| </ul> |
| </div> |
| <div class="panel"> |
| <ul> |
| <li> |
| <a href="/zh/what-is-flink/security">Security</a--> |
| </li> |
| <li> |
| <a href="https://www.apache.org/foundation/sponsorship.html">Donate</a> |
| </li> |
| <li> |
| <a href="https://www.apache.org/foundation/thanks.html">Thanks</a> |
| </li> |
| </ul> |
| </div> |
| <div class="panel icons"> |
| <div> |
| <a href="/posts"> |
| <div class="icon flink-blog-icon"></div> |
| <span>Flink blog</span> |
| </a> |
| </div> |
| <div> |
| <a href="https://github.com/apache/flink"> |
| <div class="icon flink-github-icon"></div> |
| <span>Github</span> |
| </a> |
| </div> |
| <div> |
| <a href="https://twitter.com/apacheflink"> |
| <div class="icon flink-twitter-icon"></div> |
| <span>Twitter</span> |
| </a> |
| </div> |
| </div> |
| </div> |
| </div> |
| |
| <hr/> |
| |
| <div class="container disclaimer"> |
| <p>The contents of this website are © 2024 Apache Software Foundation under the terms of the Apache License v2. Apache Flink, Flink, and the Flink logo are either registered trademarks or trademarks of The Apache Software Foundation in the United States and other countries.</p> |
| </div> |
| |
| |
| |
| </footer> |
| |
| </body> |
| </html> |
| |
| |
| |
| |
| |
| |
