All, It seems that we've implicitly picked review then commit (RTC) instead of commit then review (CTR). Apache projects allow either approach and I'm fine with either. We should just state what we are doing. I'd also like to propose that we keep a CHANGES.txt file that includes who contributed and committed each patch. I've created GIRAPH-19 to do that.
Thoughts? -- Owen