> On 2011-11-14 07:25:55, Mohammad Islam wrote:
> > Pls add a release.log entry.
> 
> Alejandro Abdelnur wrote:
>     Wouldn't make more sense (as it is done in Hadoop as well) that the 
> committer edits the release log at commit time?
>     
>     This would avoid unnecessary conflicts when patches are committed out of 
> order.

So we can adopt the same policy as followed in hadoop. No release log entry is 
needed. The committer will add one.
I agree it will reduce the # of conflicts.


- Mohammad


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2802/#review3202
-----------------------------------------------------------


On 2011-11-15 06:48:22, Alejandro Abdelnur wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/2802/
> -----------------------------------------------------------
> 
> (Updated 2011-11-15 06:48:22)
> 
> 
> Review request for oozie.
> 
> 
> Summary
> -------
> 
> The patch loads all services classes, initializes them to find out the 
> service interface and keeps the latest one, after that only the final ones 
> are initialized.
> 
> This will address the problem of the 
> HadoopAccessorService/KerberosAccessorService
> 
> 
> This addresses bug OOZIE-480.
>     https://issues.apache.org/jira/browse/OOZIE-480
> 
> 
> Diffs
> -----
> 
>   /trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1202059 
>   /trunk/core/src/main/java/org/apache/oozie/service/Services.java 1202059 
>   /trunk/core/src/test/java/org/apache/oozie/service/TestServices.java 
> 1202059 
> 
> Diff: https://reviews.apache.org/r/2802/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alejandro
> 
>

Reply via email to