Details
-
Improvement
-
Status: Open
-
Major
-
Resolution: Unresolved
-
2.7.0
-
None
-
None
Description
We don't have a documented style guide for the Hadoop source or its tests other than "use the Java rules with two spaces".
That doesn't cover policy like
- exception handling
- logging
- metrics
- what makes a good test
- why features that have O(n) or worse complexity or extra memory load on the NN & RM Are "unwelcome",
- ... etc
We have those in our heads, and we reject patches for not following them —but as they aren't written down, how can we expect new submitters to follow them, or back up our vetos with a policy to point at.
I propose having an up to date style guide which defines the best practises we expect for new codes. That can be stricter than the existing codebase: we want things to improve.
Attachments
Issue Links
- relates to
-
HADOOP-12145 Organize and update CodeReviewChecklist wiki
- Open