> On 2012-01-25 19:46:08, Mohammad Islam wrote: > > Good work. > > What about the backward compatibility? if some system is using the old one, > > how will that work. > > Harsh J wrote: > Good point, I'll throw in a config fallback, with a note to remove by > next release. Will upload patch by tomorrow as its EOD for me here.
Harsh, will you be uploading a new diff? - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3626/#review4600 ----------------------------------------------------------- On 2012-01-25 16:49:54, Harsh J wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3626/ > ----------------------------------------------------------- > > (Updated 2012-01-25 16:49:54) > > > Review request for oozie. > > > Summary > ------- > > Currently user has to define "oozie.servlet.CallbackServlet.max.data.len" to > override the default value of 2K. > This is misnomer. It should be 'oozie.action.max.output.data'. > Relevant discussion is: > http://tech.groups.yahoo.com/group/Oozie-users/message/1812 > > > This addresses bug OOZIE-4. > https://issues.apache.org/jira/browse/OOZIE-4 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 1235746 > > http://svn.apache.org/repos/asf/incubator/oozie/trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java > 1235746 > > Diff: https://reviews.apache.org/r/3626/diff > > > Testing > ------- > > > Thanks, > > Harsh > >
