Re: Review Request 67142: Add SlaManager to encapsulate SLA operations.

2018-05-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67142/#review203174 --- Master (805a53f) is red with this patch.

Review Request 67142: Add SlaManager to encapsulate SLA operations.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67142/ --- Review request for Aurora, David McLaughlin, Jordan Ly, Renan DelValle, and

Re: Review Request 67141: Introduce structs to enable specifying custom SLA.

2018-05-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67141/#review203167 --- Master (805a53f) is red with this patch.

Re: Review Request 67141: Introduce structs to enable specifying custom SLA.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67141/ --- (Updated May 15, 2018, 2:15 p.m.) Review request for Aurora, David McLaughlin,

Review Request 67141: Introduce structs to enable specifying custom SLA.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67141/ --- Review request for Aurora. Repository: aurora Description --- Add

Re: Review Request 66716: Enable `Tasks` to specify their own custom maintenance SLA.

2018-05-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/#review203153 --- Master (805a53f) is red with this patch.

Re: Review Request 66716: Enable `Tasks` to specify their own custom maintenance SLA.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/#review203150 --- @ReviewBot retry - Santhosh Kumar Shanmugham On May 15, 2018,

Re: Review Request 66716: Enable `Tasks` to specify their own custom maintenance SLA.

2018-05-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/#review203149 --- Master (805a53f) is red with this patch.

Re: Review Request 66716: Enable `Tasks` to specify their own custom maintenance SLA.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/ --- (Updated May 15, 2018, 10:16 a.m.) Review request for Aurora, David

Re: Review Request 66716: [WIP] Enable `Tasks` to specify their own custom maintenance SLA.

2018-05-15 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/ --- (Updated May 15, 2018, 10:15 a.m.) Review request for Aurora, David

Re: Review Request 66192: [WIP] Variable group size updates

2018-05-15 Thread Santhosh Kumar Shanmugham
> On May 14, 2018, 10:58 a.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > > Lines 826 (patched) > > > > > > Sums up to be exactly

Re: Review Request 66192: [WIP] Variable group size updates

2018-05-15 Thread Santhosh Kumar Shanmugham
> On May 14, 2018, 7:26 p.m., Aurora ReviewBot wrote: > > Master (805a53f) is red with this patch. > > ./build-support/jenkins/build.sh > > > > :compileJmhJavaNote: > > /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/fakes/FakeSchedulerDriver.java > >

Re: Review Request 66192: [WIP] Variable group size updates

2018-05-15 Thread Renan DelValle
> On May 14, 2018, 10:58 a.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > > Lines 826 (patched) > > > > > > Sums up to be exactly

Re: Review Request 66192: [WIP] Variable group size updates

2018-05-15 Thread Renan DelValle
> On May 14, 2018, 7:26 p.m., Aurora ReviewBot wrote: > > Master (805a53f) is red with this patch. > > ./build-support/jenkins/build.sh > > > > :compileJmhJavaNote: > > /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/fakes/FakeSchedulerDriver.java > >

Re: Review Request 66192: [WIP] Variable group size updates

2018-05-15 Thread Santhosh Kumar Shanmugham
> On May 14, 2018, 7:26 p.m., Aurora ReviewBot wrote: > > Master (805a53f) is red with this patch. > > ./build-support/jenkins/build.sh > > > > :compileJmhJavaNote: > > /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/fakes/FakeSchedulerDriver.java > >