Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Santhosh Kumar Shanmugham
> On April 21, 2017, 9:57 a.m., Aurora ReviewBot wrote: > > Master (b847db8) is green with this patch. > > ./build-support/jenkins/build.sh > > > > I will refresh this build result if you post a review containing > > "@ReviewBot retry" We have decided that this patch is not necessary and

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172676 --- Ship it! Master (b847db8) is green with this patch.

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Reza Motamedi via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172674 --- Ship it! Ship It! - Reza Motamedi On April 21, 2017, 4:32

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172673 --- Ship it! Ship It! - Stephan Erb On April 21, 2017, 6:32

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/ --- (Updated April 21, 2017, 9:32 a.m.) Review request for Aurora, Stephan Erb and

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172659 --- Ship it! Master (b847db8) is green with this patch.

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172658 --- @ReviewBot retry - Stephan Erb On April 21, 2017, 9:58 a.m.,

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Stephan Erb
> On April 21, 2017, 2:35 p.m., Aurora ReviewBot wrote: > > Master (b847db8) is red with this patch. > > ./build-support/jenkins/build.sh > > > > Test coverage missing for > > org/apache/aurora/scheduler/storage/log/SnapshotStoreImpl$8 > > Test coverage missing for > >

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172642 --- Master (b847db8) is red with this patch.

Re: Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/#review172620 --- Please mention this in the changelog. - Stephan Erb On April

Review Request 58605: Default watch_secs to 0.

2017-04-21 Thread Santhosh Kumar Shanmugham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58605/ --- Review request for Aurora, Stephan Erb and Zameer Manji. Repository: aurora