----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41678/#review111752 -----------------------------------------------------------
Ship it! Ship It! - pavan kumar kolamuri On Dec. 23, 2015, 7:30 a.m., Pallavi Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41678/ > ----------------------------------------------------------- > > (Updated Dec. 23, 2015, 7:30 a.m.) > > > Review request for Falcon. > > > Bugs: FALCON-1679 > https://issues.apache.org/jira/browse/FALCON-1679 > > > Repository: falcon-git > > > Description > ------- > > Added an additional flag (-showScheduler) to status. This will tell on which > scheduler the entity is/was scheduled. > > > Diffs > ----- > > client/src/main/java/org/apache/falcon/cli/FalconEntityCLI.java cb39a2f > client/src/main/java/org/apache/falcon/client/AbstractFalconClient.java > 43a342d > client/src/main/java/org/apache/falcon/client/FalconClient.java aea39a6 > prism/src/main/java/org/apache/falcon/resource/AbstractEntityManager.java > 5a6d2dc > > prism/src/main/java/org/apache/falcon/resource/proxy/SchedulableEntityManagerProxy.java > d3ba189 > prism/src/test/java/org/apache/falcon/aspect/LoggingAspectTest.java ee60f4f > unit/src/main/java/org/apache/falcon/unit/FalconUnitClient.java e86e3e8 > > unit/src/main/java/org/apache/falcon/unit/LocalSchedulableEntityManager.java > d30b028 > unit/src/test/java/org/apache/falcon/unit/TestFalconUnit.java 68c09d5 > > webapp/src/main/java/org/apache/falcon/resource/SchedulableEntityManager.java > 4a20e41 > > webapp/src/test/java/org/apache/falcon/resource/AbstractSchedulerManagerJerseyIT.java > 5001fe6 > webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java > 7157766 > > webapp/src/test/java/org/apache/falcon/resource/EntitySchedulerManagerJerseyIT.java > 67a51d9 > > Diff: https://reviews.apache.org/r/41678/diff/ > > > Testing > ------- > > Modified existing ITs to additionally validate this. Manual testing done. > > > Thanks, > > Pallavi Rao > >
