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




ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
 (line 267)
<https://reviews.apache.org/r/43049/#comment178477>

    Can we log if this exists?
    We already have a lot of conditions in the not_if



ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
 (line 269)
<https://reviews.apache.org/r/43049/#comment178472>

    The ls command will only list the file names. This  should also contain the 
size in bytes and date modified, remove blank lines, and should be sorted.
    
    ls -l {oozie_libext_dir} | awk '{print $9, $5, $6, $7, $8}' | awk 'NF > 0' 
| sort


- Alejandro Fernandez


On Feb. 1, 2016, 2:03 p.m., Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43049/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2016, 2:03 p.m.)
> 
> 
> Review request for Ambari and Dmitro Lisnichenko.
> 
> 
> Bugs: AMBARI-14863
>     https://issues.apache.org/jira/browse/AMBARI-14863
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
> server/libext  
> Oozie war should be updated.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py
>  24a35ad 
>   ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 
> 2183425 
> 
> Diff: https://reviews.apache.org/r/43049/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>

Reply via email to