Patch review process

[Colm O hEigeartaigh <co...@apache.org>]

Hi all,

I've raised concerns before about the length of time it's taking to review
patches. For example, this is now waiting for 5 weeks for a review:

https://reviews.apache.org/r/51519/

If we want to graduate from the Incubator this needs to improve.

Perhaps we could weaken the RTC policy for committers for changes that only
affect tests or are considered to be relatively trivial?

Colm.


-- 
Colm O hEigeartaigh

Talend Community Coder
http://coders.talend.com

Re: Patch review process

[Velmurugan Periasamy <vp...@hortonworks.com>]

Colm:

I agree with you that review process needs to be made faster.

However my opinion is to keep RTC policy as is.

Thank you,
Vel

From: Colm O hEigeartaigh <co...@apache.org>>
Reply-To: "dev@ranger.incubator.apache.org<ma...@ranger.incubator.apache.org>" <de...@ranger.incubator.apache.org>>, "coheigea@apache.org<ma...@apache.org>" <co...@apache.org>>
Date: Tuesday, October 11, 2016 at 9:09 AM
To: "dev@ranger.incubator.apache.org<ma...@ranger.incubator.apache.org>" <de...@ranger.incubator.apache.org>>
Subject: Patch review process

Hi all,

I've raised concerns before about the length of time it's taking to review
patches. For example, this is now waiting for 5 weeks for a review:

https://reviews.apache.org/r/51519/

If we want to graduate from the Incubator this needs to improve.

Perhaps we could weaken the RTC policy for committers for changes that only
affect tests or are considered to be relatively trivial?

Colm.


--
Colm O hEigeartaigh

Talend Community Coder
http://coders.talend.com