LoginSignup
1

More than 5 years have passed since last update.

Hadoop 開発の code reviewer になりたいのですが,どうすればなれるのでしょうか.

Last updated at Posted at 2013-04-15

質問

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

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
1