Re: Review Request 50168: Add rollback functionality to the scheduler

2016-08-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50168/#review145289 --- Ship it! Master (581262c) is green with this patch.

Re: Review Request 50168: Add rollback functionality to the scheduler

2016-08-09 Thread Igor Morozov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50168/ --- (Updated Aug. 9, 2016, 10:41 p.m.) Review request for Aurora, Joshua Cohen,

Re: Review Request 50931: Remove unnecessary guice container parameters.

2016-08-09 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50931/#review145300 --- Ship it! Ship It! - Joshua Cohen On Aug. 9, 2016, 7:53

Re: Review Request 50937: Bump jetty dependency to the latest release.

2016-08-09 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50937/#review145301 --- Ship it! Ship It! - Joshua Cohen On Aug. 9, 2016, 9:57

Re: Review Request 50168: Add rollback functionality to the scheduler

2016-08-09 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50168/#review145302 --- Ship it! Thanks for adding that transition. This looks good to

Review Request 50931: Remove unnecessary guice container parameters.

2016-08-09 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50931/ --- Review request for Aurora and Joshua Cohen. Repository: aurora Description

Re: Review Request 50931: Remove unnecessary guice container parameters.

2016-08-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50931/#review145261 --- Master (581262c) is green with this patch.

Re: Review Request 50931: Remove unnecessary guice container parameters.

2016-08-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50931/#review145256 --- Master (581262c) is red with this patch.

Re: Review Request 50931: Remove unnecessary guice container parameters.

2016-08-09 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50931/#review145258 --- @ReviewBot retry - Zameer Manji On Aug. 9, 2016, 12:53 p.m.,

Review Request 50937: Bump jetty dependency to the latest release.

2016-08-09 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50937/ --- Review request for Aurora and Joshua Cohen. Repository: aurora Description

Re: Review Request 50937: Bump jetty dependency to the latest release.

2016-08-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50937/#review145279 --- Ship it! Master (581262c) is green with this patch.

Re: Review Request 50168: Add rollback functionality to the scheduler

2016-08-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50168/#review145283 --- This patch does not apply cleanly against master (581262c), do

Re: Review Request 50168: Add rollback functionality to the scheduler

2016-08-09 Thread Igor Morozov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50168/ --- (Updated Aug. 9, 2016, 10:31 p.m.) Review request for Aurora, Joshua Cohen,