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



prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java (line 
584)
<https://reviews.apache.org/r/37749/#comment151631>

    If you are using only the keys.. you might as well just use 
filterByFieldsValues.keySet()


- Pallavi Rao


On Aug. 25, 2015, 6:39 a.m., Narayan Periwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37749/
> -----------------------------------------------------------
> 
> (Updated Aug. 25, 2015, 6:39 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> falcon entity -list  -type process    -filterBy 
> STATUS:RUNNING,CLUSTER:lhr1-emerald
> (PROCESS) FETL-Enhancer-Deferred
> (PROCESS) FETL-Enhancer
> (PROCESS) FETL-ConversionRC-conversion
> 
> Currently, entity list outputs only the name. Adding the filters that are 
> specified in the filterBy, to the list output, will be useful to the user.
> 
> 
> Diffs
> -----
> 
>   prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java 
> f2f9826 
> 
> Diff: https://reviews.apache.org/r/37749/diff/
> 
> 
> Testing
> -------
> 
> Done
> 
> 
> Thanks,
> 
> Narayan Periwal
> 
>

Reply via email to