質問
Hadoop には,パッチが大量にありますが,Review が間に合っていないパッチが多くあるように感じます. その問題を解決し,開発を加速するために,code reviewerになりたいと考えているのですが,どういった手順でなれるのでしょうか.
回答
code reviewer になるための条件は、コントリビュータでさえあれば特に制限はありません。
コミッター達がレビューすることが多いのですが、それはあくまで慣習的なものです。レビューされずに放置されているJIRAは本当に多いので、レビュワーが増えることは大歓迎です!
レビューの手順は簡単で、
- レビューします。Hadoop コミュニティではあまり review board を使わないですが、使うことも可能です。HBase コミュニティではかなり高い頻度で review board を使います。Hive では Phabricator が使われます。
- レビューが完了したら JIRA を編集し、 reviewed にチェックをつけます。
レビューの観点
レビューの観点についての明確な定義はありませんが、以下の事項を指摘することが多いです。
- 変更の必要性
- コード変更による影響範囲
- 設計
- 命名規則
- インデント
- イディオムの統一
- テストの有無
- コードの重複
レビューを行う上で自分が分からないこと、決定できないことについては、その分野のコミッタに相談するのも良いでしょう。相談する際には,Hadoop の JIRA のコメント欄についている mention 機能を利用すると便利です。mention 機能の使い方は以下の通りです:
- JIRA のコメント欄で @[ユーザ名] と入力する.
- ユーザが複数候補に挙がるので、 mention を送りたい方の名前を選択します.
- コメント欄に [~ユーザ名]という名前が補完されます.これで mention 完了です.
あるユーザに mention すると、JIRA に登録されているメールアドレスに "mention されているよ" という意味のメールが送付されるようになっています。
注意
この投稿は、Hadoop Hackathon in Tokyo の QA サイト閉鎖に伴う移行作業により作成されたものです。
移行元URL: http://glowing-moon-7493.herokuapp.com/questions/471/code-reviewer