----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8346/ -----------------------------------------------------------
(Updated Dec. 14, 2012, 3:15 a.m.) Review request for oozie. Changes ------- Incorporated Virag's review comments - added null check and subsequent instantiation of new connection much before in the logic - removed list of callables and instead directly queue as callable Description ------- https://issues.apache.org/jira/browse/OOZIE-1108 patch *on TOP* of OOZIE-1111 - the HCat URI changes. This patch also fixes a lot of affected test cases This addresses bug OOZIE-1108. https://issues.apache.org/jira/browse/OOZIE-1108 Diffs (updated) ----- branches/hcat-intre/core/src/main/java/org/apache/oozie/ErrorCode.java 1421635 branches/hcat-intre/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdatePushMissingDependency.java 1421635 branches/hcat-intre/core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java 1421635 branches/hcat-intre/core/src/main/java/org/apache/oozie/jms/MessageReceiver.java 1421635 branches/hcat-intre/core/src/main/java/org/apache/oozie/service/JMSAccessorService.java 1421635 branches/hcat-intre/core/src/main/java/org/apache/oozie/service/PartitionDependencyManagerService.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/command/coord/TestCoordActionUpdatePushMissingDependency.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/jms/TestHCatMessageHandler.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/jms/TestMessageReceiver.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/service/TestPartitionDependencyManagerService.java 1421635 branches/hcat-intre/core/src/test/java/org/apache/oozie/test/XTestCase.java 1421635 Diff: https://reviews.apache.org/r/8346/diff/ Testing ------- yes Thanks, Mona Chitnis
