Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-20 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149678 --- Ship it! Ship It! - Stephan Erb On Sept. 19, 2016, 8:28 p.m.

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-20 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149674 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 19, 2016, 6:2

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149547 --- Ship it! Ship It! - Zameer Manji On Sept. 19, 2016, 11:28 a.

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149539 --- Ship it! Master (4c4040f) is green with this patch. ./build-s

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149530 --- @ReviewBot retry - Dmitriy Shirchenko On Sept. 19, 2016, 6:28 p

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149527 --- Master (330e8ee) is red with this patch. ./build-support/jenkins

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/ --- (Updated Sept. 19, 2016, 6:28 p.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-19 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149507 --- src/main/java/org/apache/aurora/scheduler/events/Webhook.java (li

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-16 Thread Dmitriy Shirchenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/ --- (Updated Sept. 17, 2016, 1:40 a.m.) Review request for Aurora, Maxim Khutornenk

Re: Review Request 51980: Refactor of Webhook and no longer posting entire task state via webhook on scheduler restart

2016-09-16 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51980/#review149311 --- Ship it! Master (a87ad41) is green with this patch. ./build-s