----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4219/#review5714 -----------------------------------------------------------
trunk/client/src/main/java/org/apache/oozie/client/WorkflowJob.java <https://reviews.apache.org/r/4219/#comment12447> pls remove this space. trunk/client/src/main/java/org/apache/oozie/client/WorkflowJob.java <https://reviews.apache.org/r/4219/#comment12448> space pls - Mohammad On 2012-03-07 09:57:47, shwethags wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4219/ > ----------------------------------------------------------- > > (Updated 2012-03-07 09:57:47) > > > Review request for oozie. > > > Summary > ------- > > getJobInfo for workflow doesn't return external id. Since oozie lets users > set it, it should be returned so that users can use it. > > > This addresses bug OOZIE-742. > https://issues.apache.org/jira/browse/OOZIE-742 > > > Diffs > ----- > > trunk/client/src/main/java/org/apache/oozie/client/WorkflowJob.java 1297889 > trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java > 1297889 > trunk/core/src/main/java/org/apache/oozie/WorkflowJobBean.java 1297889 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowsJobGetJPAExecutor.java > 1297889 > > trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowJobGetJPAExecutor.java > 1297889 > trunk/core/src/test/java/org/apache/oozie/test/XDataTestCase.java 1297889 > > Diff: https://reviews.apache.org/r/4219/diff > > > Testing > ------- > > UT - TestWorkflowJobGetJPAExecutor > > > Thanks, > > shwethags > >
