It would be good to have a policy for submitting a new patch for review.
If the patch is small, usually it is pretty easy to review.But, if it large,
a GUI like reviewboard (https://reviews.apache.org) makes it easy.
So, going forward, I would like to propose either of the following.
1. All patches must go through reviewboard
2. If a contributor/reviewer creates a reviewboard request,
all subsequent review requests should go through the reviewboard.
I would personally vote for 2., since for small patches, we don’t really need a
reviewboard.
But, please vote, and based on that, we can come up with a policy.
Let us know, if you think of some other option.
Thanks,
-namit