> On Sept. 8, 2016, 9:32 a.m., Rajat Khandelwal wrote: > > lens-server/src/main/java/org/apache/lens/server/scheduler/SchedulerServiceImpl.java, > > line 479 > > <https://reviews.apache.org/r/51457/diff/1/?file=1486693#file1486693line479> > > > > Log seems misplaced. We're not killing here, right?
Yes, we are killing the instance. Because there is no query to kill, we can just update the status and move on. - Lavkesh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51457/#review148004 ----------------------------------------------------------- On Aug. 26, 2016, 11:28 a.m., Lavkesh Lahngir wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51457/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2016, 11:28 a.m.) > > > Review request for lens. > > > Bugs: LENS-1286 > https://issues.apache.org/jira/browse/LENS-1286 > > > Repository: lens > > > Description > ------- > > When the server is restarted, it reads all instances which where in WAITING, > LAUNCHED or RUNNING states. > If the instance is in WAITING or LAUNCHED state, it is killed and it is rerun. > If the instance is in RUNNING state, it will check the query status and > update accordingly. > > P.S. Ignore ResourceTest file. It is in separate review request. > > > Diffs > ----- > > > lens-api/src/main/java/org/apache/lens/api/scheduler/SchedulerJobInstanceRun.java > 8532ed0 > > lens-server/src/main/java/org/apache/lens/server/scheduler/SchedulerDAO.java > 966a64e > > lens-server/src/main/java/org/apache/lens/server/scheduler/SchedulerQueryEventListener.java > 077d531 > > lens-server/src/main/java/org/apache/lens/server/scheduler/SchedulerServiceImpl.java > 323e053 > > lens-server/src/test/java/org/apache/lens/server/scheduler/ScheduleResourceTest.java > PRE-CREATION > > lens-server/src/test/java/org/apache/lens/server/scheduler/SchedulerRestartTest.java > PRE-CREATION > > lens-server/src/test/java/org/apache/lens/server/scheduler/TestSchedulerServiceImpl.java > 130df5f > > lens-server/src/test/java/org/apache/lens/server/scheduler/util/SchedulerTestUtils.java > a36b2aa > > Diff: https://reviews.apache.org/r/51457/diff/ > > > Testing > ------- > > > Thanks, > > Lavkesh Lahngir > >
