Re: Review Request 55105: AURORA-1870 Add finer grained timings to the Snapshot process

2017-01-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55105/#review161021 --- Ship it! Master (d4ebb56) is green with this patch.

Re: Review Request 55105: AURORA-1870 Add finer grained timings to the Snapshot process

2017-01-09 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55105/ --- (Updated Jan. 9, 2017, 5:11 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 55347: Ensure destination exists when mounting files into a filesystem image.

2017-01-09 Thread Joshua Cohen
> On Jan. 9, 2017, 10:57 p.m., Santhosh Kumar Shanmugham wrote: > > src/main/python/apache/aurora/executor/common/sandbox.py, lines 285-291 > > > > > > I tried a `mkdir` followed by `touch` and the path still

Re: Review Request 55284: Benchmarking

2017-01-09 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55284/#review161001 --- Ship it!

Re: Review Request 55347: Ensure destination exists when mounting files into a filesystem image.

2017-01-09 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55347/#review160995 --- src/main/python/apache/aurora/executor/common/sandbox.py (lines

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Mehrdad Nurolahzade
> On Jan. 9, 2017, 1:45 p.m., Aurora ReviewBot wrote: > > Master (d4ebb56) is red with this patch. > > ./build-support/jenkins/build.sh > > > > at > > org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120) > > at > >

Re: Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/#review160974 --- Master (d4ebb56) is red with this patch.

Review Request 55357: AURORA-1867 Consider reserving for multiple tasks per preemption round

2017-01-09 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55357/ --- Review request for Aurora, David McLaughlin, Joshua Cohen, Stephan Erb, and

Re: Review Request 55347: Ensure destination exists when mounting files into a filesystem image.

2017-01-09 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55347/#review160933 --- src/main/python/apache/aurora/executor/common/sandbox.py (lines

Re: Review Request 55347: Ensure destination exists when mounting files into a filesystem image.

2017-01-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55347/#review160912 --- Ship it! Master (d4ebb56) is green with this patch.

Review Request 55347: Ensure destination exists when mounting files into a filesystem image.

2017-01-09 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55347/ --- Review request for Aurora, Santhosh Kumar Shanmugham and Stephan Erb.

Re: Review Request 55105: AURORA-1870 Add finer grained timings to the Snapshot process

2017-01-09 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55105/#review160905 ---

Re: Review Request 55284: Benchmarking

2017-01-09 Thread Mehrdad Nurolahzade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55284/#review160900 ---