----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14129/#review26092 -----------------------------------------------------------
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java <https://reviews.apache.org/r/14129/#comment50927> catch should be in a new line. Please use the formatting from https://cwiki.apache.org/confluence/pages/viewpageattachments.action?pageId=30737371 http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java <https://reviews.apache.org/r/14129/#comment50928> if blocks should have { } http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/util/GraphGenerator.java <https://reviews.apache.org/r/14129/#comment50929> actionsLimit. space before and after = http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/util/GraphGenerator.java <https://reviews.apache.org/r/14129/#comment50930> Don't need to declare exception message as a string. Just use that in the exception constructor directly http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/util/TestGraphGenerator.java <https://reviews.apache.org/r/14129/#comment50933> This test case is enough. Can remove the others. testWrite() will take care of those http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/util/TestGraphGenerator.java <https://reviews.apache.org/r/14129/#comment50934> graphWF_<n>_actions.xml Simplify the graphWF file actions. http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/oozie-console.js <https://reviews.apache.org/r/14129/#comment50937> function alertOnDAGError() http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/oozie-console.js <https://reviews.apache.org/r/14129/#comment50935> space before and after http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/oozie-console.js <https://reviews.apache.org/r/14129/#comment50936> &token=Math.random is not required http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/oozie-console.js <https://reviews.apache.org/r/14129/#comment50938> Remove this line and fetchDAG method - Rohini Palaniswamy On Sept. 13, 2013, 6:57 p.m., Purshotam Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14129/ > ----------------------------------------------------------- > > (Updated Sept. 13, 2013, 6:57 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1529 > https://issues.apache.org/jira/browse/OOZIE-1529 > > > Repository: oozie > > > Description > ------- > > 1.Disabling job DAG display for workflow having action > 25. > Job DAQ rendering happens at server side. It's a heavy operation and server > goes OOM while generating DAG for workflow having more actions. > > 2. There are other UI fixes. > a.)Client calls to generates DAG two times, 1> while initializing panel. 2> > When user clicks Job DAG tab. > Disabling the DAG generation while initializing panel. It will be called > only when user clicks Job DAG panel. > > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java > 1520065 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/util/GraphGenerator.java > 1520065 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/util/TestGraphGenerator.java > 1520065 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/resources/graphWF_equal.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/resources/graphWF_huge_state.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/oozie-console.js > 1520065 > > Diff: https://reviews.apache.org/r/14129/diff/ > > > Testing > ------- > > Added three test case. > 1. If actions<25, should generate graph > 2. If actions=25, should generate graph > 3. If actions>25, should throw error message. > > > Thanks, > > Purshotam Shah > >
