> On Jan. 24, 2013, 11:20 p.m., Virag Kothari wrote: > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/jms/JMSExceptionListener.java, > > line 27 > > <https://reviews.apache.org/r/9061/diff/2/?file=251720#file251720line27> > > > > javadoc here and for other public stuff
Done. Did not do for methods with @Override as it is not required. > On Jan. 24, 2013, 11:20 p.m., Virag Kothari wrote: > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/service/SchedulerService.java, > > line 188 > > <https://reviews.apache.org/r/9061/diff/2/?file=251722#file251722line188> > > > > Can we reuse the existing methods of Scheduler SErvice No. Want to schedule only once after a delay. Existing methods do it repeatedly based on a interval - Rohini ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9061/#review15659 ----------------------------------------------------------- On Jan. 25, 2013, 12:03 a.m., Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9061/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2013, 12:03 a.m.) > > > Review request for oozie. > > > Description > ------- > > Retry jms connections on failure backing off exponentially (1min,2 min, > 4,8,16..) up to a max limit > > > This addresses bug OOZIE-1185. > https://issues.apache.org/jira/browse/OOZIE-1185 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/dependency/FSURIHandler.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/dependency/HCatURIHandler.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/jms/DefaultConnectionContext.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/jms/JMSExceptionListener.java > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/service/JMSAccessorService.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/service/SchedulerService.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/test/java/org/apache/oozie/test/MiniHCatServer.java > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/hcataloglibs/hcatalog-0.6/pom.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/hcataloglibs/pom.xml > 1438270 > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/pom.xml 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/src/main/assemblies/hcataloglib.xml > 1438270 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/src/main/assemblies/hcataloglibs.xml > 1438270 > > Diff: https://reviews.apache.org/r/9061/diff/ > > > Testing > ------- > > > Thanks, > > Rohini Palaniswamy > >