----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10355/#review19412 -----------------------------------------------------------
trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java <https://reviews.apache.org/r/10355/#comment40136> the two else if's doesn't seem required trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java <https://reviews.apache.org/r/10355/#comment40137> doesn't seem required - Virag Kothari On April 8, 2013, 11:04 p.m., Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10355/ > ----------------------------------------------------------- > > (Updated April 8, 2013, 11:04 p.m.) > > > Review request for oozie. > > > Description > ------- > > Bulk monitoring API is to be extended for use via CLI. > - also edited earlier code to remove unused json settings > - added doc in CommandLineTool.twiki > - added appropriate unit tests > - tested the real output by building distro and running examples > > > This addresses bug OOZIE-1303. > https://issues.apache.org/jira/browse/OOZIE-1303 > > > Diffs > ----- > > trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1465733 > trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java > 1465733 > trunk/client/src/test/java/org/apache/oozie/client/rest/TestJsonToBean.java > 1465733 > trunk/core/src/main/java/org/apache/oozie/client/rest/BulkResponseImpl.java > 1465733 > trunk/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java > 1465733 > trunk/docs/src/site/twiki/DG_CommandLineTool.twiki 1465733 > > Diff: https://reviews.apache.org/r/10355/diff/ > > > Testing > ------- > > yes > > > Thanks, > > Mona Chitnis > >
