----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14129/#review26111 -----------------------------------------------------------
Ship it! Ship It! - Rohini Palaniswamy On Sept. 13, 2013, 10:48 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, 10:48 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_26_actions.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 > >
