-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38088/
-----------------------------------------------------------
(Updated Sept. 9, 2015, 12:50 p.m.)
Review request for oozie, Purshotam Shah and Robert Kanter.
Changes
-------
Added oozie job console url to the test
Bugs: OOZIE-2346
https://issues.apache.org/jira/browse/OOZIE-2346
Repository: oozie-git
Description
-------
Add super parent id and workflow depth information along with the other oozie
job information into the job conf.
Diffs (updated)
-----
core/src/main/java/org/apache/oozie/action/hadoop/OozieJobInfo.java 4b13daa
core/src/main/java/org/apache/oozie/action/oozie/SubWorkflowActionExecutor.java
854d621
core/src/test/java/org/apache/oozie/action/hadoop/TestOozieJobInfo.java
ceaef8b
Diff: https://reviews.apache.org/r/38088/diff/
Testing
-------
Result of running test-patch locally: bin/test-patch --JIRA=OOZIE-2346
--reset-scm
+1 PATCH_APPLIES
+1 CLEAN
+1 RAW_PATCH_ANALYSIS
+1 the patch does not introduce any @author tags
+1 the patch does not introduce any tabs
+1 the patch does not introduce any trailing spaces
+1 the patch does not introduce any line longer than 132
+1 the patch does adds/modifies 1 testcase(s)
+1 RAT
+1 the patch does not seem to introduce new RAT warnings
+1 JAVADOC
+1 the patch does not seem to introduce new Javadoc warnings
WARNING: the current HEAD has 91 Javadoc warning(s)
+1 COMPILE
+1 HEAD compiles
+1 patch compiles
+1 the patch does not seem to introduce new javac warnings
+1 BACKWARDS_COMPATIBILITY
+1 the patch does not change any JPA Entity/Colum/Basic/Lob/Transient
annotations
+1 the patch does not modify JPA files
-1 TESTS
Tests run: 1689
Tests failed: 3
Tests errors: 0
The patch failed the following testcases:
testHostname(org.apache.oozie.servlet.TestHostnameFilter)
testStreaming(org.apache.oozie.action.hadoop.TestMapReduceActionExecutor)
testStreamingConfOverride(org.apache.oozie.action.hadoop.TestMapReduceActionExecutor)
+1 DISTRO
+1 distro tarball builds with the patch
----------------------------
-1 Overall result, please check the reported -1(s)
The test failures are not related to the fix.
Thanks,
Akshay Rai
