-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13961/
-----------------------------------------------------------
(Updated Sept. 24, 2013, 7:34 a.m.)
Review request for oozie.
Changes
-------
addressed Ryota's comments and fixed failing tests
Bugs: OOZIE-1503
https://issues.apache.org/jira/browse/OOZIE-1503
Repository: oozie
Description
-------
Patch to revisit eagerLoadState calls causing multiple DB calls, at times
redundant, and to load only select columns for eagerLoadState where applicable
Diffs (updated)
-----
trunk/core/src/main/java/org/apache/oozie/BundleJobBean.java 1525636
trunk/core/src/main/java/org/apache/oozie/CoordinatorJobBean.java 1525636
trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1525636
trunk/core/src/main/java/org/apache/oozie/WorkflowActionBean.java 1525636
trunk/core/src/main/java/org/apache/oozie/WorkflowJobBean.java 1525636
trunk/core/src/main/java/org/apache/oozie/command/RerunTransitionXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobChangeXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobResumeXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobSuspendXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleKillXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleRerunXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleStartXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleStatusUpdateXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleSubmitXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionStartXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordRerunXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/ActionXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/command/wf/WorkflowXCommand.java
1525636
trunk/core/src/main/java/org/apache/oozie/executor/jpa/BundleActionQueryExecutor.java
1525636
trunk/core/src/main/java/org/apache/oozie/executor/jpa/BundleJobQueryExecutor.java
1525636
trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobQueryExecutor.java
1525636
trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionQueryExecutor.java
1525636
trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobQueryExecutor.java
1525636
trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordRerunXCommand.java
1525636
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestBundleActionQueryExecutor.java
PRE-CREATION
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestBundleJobQueryExecutor.java
PRE-CREATION
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobQueryExecutor.java
1525636
trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestWorkflowActionQueryExecutor.java
1525636
Diff: https://reviews.apache.org/r/13961/diff/
Testing
-------
Thanks,
Mona Chitnis