Re: Review Request 67077: Remove resource properties from ResourceAggregate

2018-05-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67077/#review202887 --- Ship it! Master (805a53f) is green with this patch.

Re: Review Request 67077: Remove resource properties from ResourceAggregate

2018-05-10 Thread Jing Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67077/ --- (Updated May 11, 2018, 3:18 a.m.) Review request for Aurora, Renan DelValle

Re: Review Request 67077: Remove resource properties from ResourceAggregate

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

Re: Review Request 67077: Remove resource properties from ResourceAggregate

2018-05-10 Thread Renan DelValle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67077/#review202882 --- @ReviewBot retry - Renan DelValle On May 10, 2018, 5:14 p.m.,

Re: Review Request 67077: Remove resource properties from ResourceAggregate

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

Review Request 67077: Remove resource properties from ResourceAggregate

2018-05-10 Thread Jing Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67077/ --- Review request for Aurora, Renan DelValle and Stephan Erb. Bugs: AURORA-1975

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

2018-05-10 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66716/#review202850 --- Looking really good! I am easily able to use this for my