Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-11-19 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review61210 --- Core logic looks great, few comments. 1. We are only supporting

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-10-17 Thread Rohini Palaniswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review57158 --- client/src/main/java/org/apache/oozie/cli/OozieCLI.java

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-09-17 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/ --- (Updated Sept. 17, 2014, 6:59 p.m.) Review request for oozie. Changes

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-09-03 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/ --- (Updated Sept. 4, 2014, 1:05 a.m.) Review request for oozie. Changes ---

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-09-03 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review52265 --- Extra line changes (false ones) in BundleSubmitX and

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-09-02 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review52110 --- client/src/main/java/org/apache/oozie/cli/OozieCLI.java

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-09-02 Thread Mona Chitnis
On Aug. 29, 2014, 4:52 p.m., Purshotam Shah wrote: client/src/main/java/org/apache/oozie/client/OozieClient.java, line 152 https://reviews.apache.org/r/24487/diff/2/?file=660982#file660982line152 Do you need to say new (newshouldend ) ? When we specify end time for

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-08-14 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/ --- (Updated Aug. 14, 2014, 11:13 p.m.) Review request for oozie. Changes

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-08-11 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review50269 --- core/src/main/resources/oozie-default.xml

Re: Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-08-08 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/#review50101 --- client/src/main/java/org/apache/oozie/cli/OozieCLI.java

Review Request 24487: OOZIE-1913 Devise a way to turn off SLA alerts for bundle/coordinator flexibly

2014-08-07 Thread Mona Chitnis
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24487/ --- Review request for oozie. Bugs: OOZIE-1913