----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4576/#review6597 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/service/LiteWorkflowStoreService.java <https://reviews.apache.org/r/4576/#comment14265> \s will match newline, space.. can you try replaceAll("\\s+", ""))? Also, will adding a test case help? - Virag On 2012-03-30 17:35:15, Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4576/ > ----------------------------------------------------------- > > (Updated 2012-03-30 17:35:15) > > > Review request for oozie. > > > Summary > ------- > > the oozie-default: > > <name>oozie.service.LiteWorkflowStoreService.user.retry.error.code</name> > <value> > JA008, > JA009, > JA017, > JA018, > JA019, > FS009, > FS008 > </value> > > but the action failed these error codes, is not retried. Retry works if this > property is specified in oozie-site.xml > > This problem is resolved by creating the error codes Set after eliminating > whitespaces if any. > > > This addresses bug OOZIE-797. > https://issues.apache.org/jira/browse/OOZIE-797 > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/service/LiteWorkflowStoreService.java > 1306604 > > Diff: https://reviews.apache.org/r/4576/diff > > > Testing > ------- > > yes > > > Thanks, > > Mona > >
