Please refer to How to Contribute for instructions how to submit a code review to http://reviews.apache.org
What makes a good code review?
- Authors should annotate source code before the review. This makes it easier for devs reviewing your code and may even help you spot bugs before they do.
- Send small code-reviews if possible. Reviewing more than 400 lines per hour diminishes our ability to find defects.
- Reviewing code for more than one hour also reduces our ability to find bugs.
- If possible, try to break up large reviews into separate but functional stages. If you need to temporarily comment out unit tests, do so. Sending gigantic patches means your review will take longer since reviewers need to block out more time to go through it, and you may spend more time revving iterations and rebasing.
We have a global community of committers, so please be mindful that you should wait at least 24 hours before pushing your patch even though you may already have the necessary +1.
This encourages others to take an interest in your review and helps us find more bugs (it's ok to slow down in order to speed up).
Always include the "Ambari" group in the code review and at least two committers that are familiar with that code area.
If you want to subscribe to code reviews for a particular area, feel free to edit this section.
Area | Example Files | Reviewers (reviewboard id) |
---|---|---|
Ambari Alerts | Alert*.java alert_*.py | jhurley |
Ambari Agent | /ambari-agent/* | |
Ambari Metrics System | /ambari-metrics/* | swagle |
Ambari Views | /ambari-views/* | |
Ambari Server (threads, caching, lifecycle) | /ambari-server/* | swagle, ncole, jhurley, afernandez |
Ambari REST API (/api/v?/....) | *Service.java | rlevas |
Blueprints | Blueprint*.java | rnettleton |
LogSearch | /ambari-logsearch/* | |
Rolling & Express Upgrade | *upgrade.xml | afernandez, ncole, jhurley, dmitriusan |
Security, Kerberos, LDAP | java files and kerberos.json | rlevas |
Stack Advisor | stack_advisor.py | srimanth |
Stack Framework | metainfo.xml, any config files | jluniya |
UI | /ambari-web/* | yusaku, rzang, jwang |