----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4064/#review5365 -----------------------------------------------------------
Ship it! +1 - Mohammad On 2012-02-27 22:19:33, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4064/ > ----------------------------------------------------------- > > (Updated 2012-02-27 22:19:33) > > > Review request for oozie. > > > Summary > ------- > > removing obsolete/not-used JobServlet and JobServlet code and correcting some > testcases to use V1 versions of the servlets > > > This addresses bug OOZIE-721. > https://issues.apache.org/jira/browse/OOZIE-721 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/servlet/JobServlet.java 1294359 > trunk/core/src/main/java/org/apache/oozie/servlet/JobsServlet.java 1294359 > trunk/core/src/test/java/org/apache/oozie/client/TestOozieCLI.java 1294359 > trunk/core/src/test/java/org/apache/oozie/client/TestWorkflowClient.java > 1294359 > trunk/core/src/test/java/org/apache/oozie/client/TestWorkflowXClient.java > 1294359 > trunk/core/src/test/java/org/apache/oozie/servlet/MockDagEngineService.java > 1294359 > trunk/core/src/test/java/org/apache/oozie/servlet/TestJobsServlet.java > 1294359 > trunk/core/src/test/java/org/apache/oozie/servlet/TestV0JobServlet.java > 1294359 > trunk/core/src/test/java/org/apache/oozie/servlet/TestV1JobsServlet.java > 1294359 > > Diff: https://reviews.apache.org/r/4064/diff > > > Testing > ------- > > run all testcases successfully with Hadoop 1.0.1 > > > Thanks, > > Alejandro > >
