1. 코드 리뷰 시스템
소개

2. 발표자 소개
● 오픈소스를 수정해서 밥벌어 먹습니다.
● 주로 임베디드 기기에서 사용되는 소프트웨
어를 개발합니다.
○ PC나 서버가 아닌 기기용 소프트웨어
○ 아파트, 지하철 충전기, 광고 패널, 티켓 발권기, 셋탑
박스, 교육용 로봇, 집 전화기, 휴대폰, 자동차, 기타
등등
● 제작년부터 코드리뷰 시스템을 써보게 되었
습니다.

3. 안드로이드 커밋 통계

4. 정말로 구글러들은 외계인?
과연 그들은
고문
당하고 있는 거신가?

5. Guido Van Rossum
● Creator of Python
● 한때 구글러
○ 2005 - 2012
● 지금은
Dropbox에
Drop 되었습니다.

6. Guido할아부지의 첫번째 미션
Mondrian이라는
코드 리뷰 시스템
을 만들었더랬죠.

7. 코드 리뷰 시스템

8. 코드 리뷰의 목적
It is intended to find and fix
MISTACKES overlooked.
defined in Wikipedia

9. 코드 리뷰의 목적
Goal is COOPERATION,
not fault-finding.
Guido van Rossum

10. 협력
코드 리뷰의 목적

11. Cooperation
Fault-finding

12. ● 뭔가 좋은거 같긴 한데
● 쓰이는데가 별로 없는 것 같애요.

13. 코드 리뷰 시스템을 도입못하
는 이유 (1)
그게 뭐하는 거임?

14. 코드 리뷰 시스템을 도입못하
는 이유 (2)
그걸 하느니 그 시간에
코드 한줄 더 짜겠심.

15. 코드 리뷰 시스템을 도입못하
는 이유 (3)
내가 뭘 하는지도 모르
겠는데, 남의 걸 어떻게
봐줌?

16. Why?
귀찮아 죽겠는데 왜 저런걸 써
라는 거지?

17. 모두가 원하는 상황

18. 하지만 현실은 시궁창.

20. 코드 리뷰 시스템

21. 코드 리뷰 시스템

22. 코드 리뷰 도입시 얻을 수 있
는 효과
● 버그 개수 감소
● 팀원의 트레이닝 도구로 활용
● 코드 가독성 증가 및 품질 상승
● 코드 세부 구현사항에 대한 기록 보존
● 팀 역량 상향평준화
● 디버깅 시간 및 프로젝트 수행 기간 단축

23. 놀라운 효과!!!

24. 놀라운 효과!!!!!!

25. 사실 코드리뷰는...
● Pair Programming
의 일종.
○ COOPERATION
○ 하지만 실시간으로 여
러명이 붙어서 작업할
필요가 없다.
○ 시간날 때 코드를 올
리고, 시간날 때 리뷰
하면 된다.

26. 코드 리뷰에 대한 오해
● 코드 리뷰는 사람이 남아도는 프로젝트에만
적합한거 아닌가요?
○ 사람 남아도는 프로젝트. 그런거 업따.
○ 서로 다른 의견을 가진 사람이 리뷰할수 있기만 하면
됨.
○ 2명만 있어도 할 수 있다.

27. 코드 리뷰에 대한 오해
● 코드 리뷰는 초천재 미소년/미소녀만 할 수 있
는거 아닌가요?
○ 지금 당신이 바로 초천재 미소년/미소녀!!
○ 못 알아보는 부분이라도 마킹해보자.
○ 앞에서도 말했지만 코드리뷰의 목적은 잘못한 부분을
찾는 것이 아니라 COOPERATION.

28. 코드 리뷰를 하는 법
● 어렵지 않아요.
○ Re(다시, 다르게)
○ View(살펴본다)

29. 사실 여러분은 코드 리뷰를 이
미 하고 있습니다.
● 버그 잡을 때
○ 버그가 있다고 생각하지 않았을 때 코드를 바라보는
관점
○ 버그가 있다고 생각하고 코드를 바라보는 관점
● 사실 관점을 다르게 볼 수 있다면 혼자서도 코
드 리뷰가 가능
● 하지만 다중인격의 소유자가 아닌 이상, 2명
이상 같이 하는게 좋습니다.

30. 코드 리뷰에 대한 오해
● 후다닥 처리해야 하는데 리뷰하면 시간만 까
먹는거 아네요?
○ 그러다가 된통 얻어맏고 야근야근 열매 섭취.
○ 코드 리뷰에 투자하는 시간은 결코 배신하지 않습니
다.

31. Gerrit

32. 잠시 살펴보는 Gerrit의 족보

33. Mondrian
● Written by Guido
van Rossum
● Written with Python
● Announced in 2006
● Integrated with
Perforce
● Hosted and Used at
Google Internal

34. RietVeld
● Written by Guido van
Rossum
● Written with Python
● Announced in 2008
● Integrated with
Subversion
● Host on Google Ap
Engine
● Used by Chrome
Project

35. Gerrit
● Gerrit (2008 - 2012)
○ fork from Rietveld
○ add Access Control
List feature
○ Written by Sean O.
Pearce
● Gerrit2 (2008 - )
○ Rewritten Gerrit with
JavaEE
○ Written by Sean O.
Pearce

36. Gerrit의 특징
● 여러가지 환경에서 운영 가능
○ jvm이 설치되어 있으면 PC에서도 운영 가능
○ 표준 servlet container 지원
● 여러가지 인증 방식 지원
○ http/ldap/openid
● 여러가지 database 지원
○ mysql/pgsql/h2

37. Gerrit의 기능
● 소스 리뷰 게시판
● Access Control List
● Git 저장소

38. Gerrit의 인터페이스
● for Human
○ WEB
● for External Integration
○ ssh
■ git
■ gerrit
○ Rest API

39. DVCS(Hg, Git) Workflow
Remote
Repository
User1's
Local
Repository
fetch
push
pull request
commit
merge
rebase
User2's
Local
Repository
fetch
push
pull request
commit
merge
rebase

40. Gerrit Workflow
Remote Git
RepositoryUser1's
Local Git
Repository
fetch
push
commit
merge
rebase
User2's
Local Git
Repository
fetch
push
commit
merge
rebase
Pending
Commit
Merge
Gerrit

41. Gerrit Detailed Workflow
push commit to
refs/for/branch
gerrit notify
reviewers
& verifiers
Online
code quality
review
IP
Review
Build
Comments
& Votes
Run testcases
improves the
change
merge the change
into master
abandons
the change
change
approved
change not
approved
flow from http://goo.gl/Kwcxo

42. Gerrit + Eclipse
● Mylyn Gerrit Connector
● Mylyn Reviews

43. DEMO

44. What's Next?
● Search Slide about Gerrit in SlideShare!
○ How Git and Gerrit make you more productive?
● Leaning git branch
● Why Git is Better then Mercurial?
● ProGit
● Repo 써보기
● Eclipse Mylyn Reviews

45. ● http://goo.gl/Z3mWx