----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37749/#review96320 -----------------------------------------------------------
prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java (line 585) <https://reviews.apache.org/r/37749/#comment151637> Can you please add a unit test also for it? See EntityManagerTest class. - Ajay Yadava 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 > >
