Re: Review Request 45286: AlarmService for the Scheduler

2016-04-22 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review130061 --- lens-server/src/test/java/org/apache/lens/server/scheduler/notifi

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-17 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review129282 --- Ship it! Ship It! - Amareshwari Sriramadasu On April 13, 201

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-13 Thread Ajay Yadava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/ --- (Updated April 13, 2016, 2:21 p.m.) Review request for lens. Changes ---

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-12 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review128371 --- lens-server/pom.xml (line 235)

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-11 Thread Ajay Yadava
> On April 3, 2016, 7:01 a.m., Raju Bairishetti wrote: > > lens-server/src/main/java/org/apache/lens/server/scheduler/notification/services/AlarmService.java, > > line 65 > > > > > > Looks like, service is not added

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-11 Thread Ajay Yadava
> On March 31, 2016, 10:52 a.m., Amareshwari Sriramadasu wrote: > > lens-server/src/main/java/org/apache/lens/server/scheduler/notification/services/AlarmService.java, > > line 83 > > > > > > Should health status be

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-11 Thread Ajay Yadava
> On March 28, 2016, 3:42 a.m., Puneet Gupta wrote: > > lens-server/src/main/java/org/apache/lens/server/LensServices.java, line 194 > > > > > > Do we need to start Alarm Service separateley or should it be started

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-11 Thread Ajay Yadava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/ --- (Updated April 11, 2016, 8:44 a.m.) Review request for lens. Bugs: LENS-987

Re: Review Request 45286: AlarmService for the Scheduler

2016-04-03 Thread Raju Bairishetti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review126715 --- lens-server/src/main/java/org/apache/lens/server/scheduler/notifi

Re: Review Request 45286: AlarmService for the Scheduler

2016-03-31 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review126292 --- lens-server-api/src/main/java/org/apache/lens/server/api/events/S

Re: Review Request 45286: AlarmService for the Scheduler

2016-03-27 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/#review125588 --- lens-server/src/main/java/org/apache/lens/server/LensServices.jav

Review Request 45286: AlarmService for the Scheduler

2016-03-24 Thread Ajay Yadava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45286/ --- Review request for lens. Bugs: LENS-987 https://issues.apache.org/jira/brow