----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15211/#review28130 -----------------------------------------------------------
Two general things: - Can you add documentation (the twiki files) for both the CLI and REST commands? - You'll need to make some changes for this to work with Oozie HA. It's nothing too complicated; I've described the basic idea in my comment on OOZIE-1519 -- let me know if you need more clarification on that. - Robert Kanter On Nov. 4, 2013, 6:51 p.m., Purshotam Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15211/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2013, 6:51 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1519 > https://issues.apache.org/jira/browse/OOZIE-1519 > > > Repository: oozie > > > Description > ------- > > Admin command to update the sharelib. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java > 1534391 > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java > 1534391 > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/client/rest/RestConstants.java > 1534391 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/BaseAdminServlet.java > 1534391 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V1AdminServlet.java > 1534391 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/client/TestOozieCLI.java > 1534391 > > Diff: https://reviews.apache.org/r/15211/diff/ > > > Testing > ------- > > > Thanks, > > Purshotam Shah > >
