Re: Review Request 52300: Running task ssh without an instance should pick a random instance

2016-09-28 Thread JING CHEN
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52300/ --- (Updated Sept. 29, 2016, 5:30 a.m.) Review request for Aurora, Joshua Cohen

Re: Review Request 52367: Upgrade to latest pants 1.2.0 pre-release.

2016-09-28 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52367/#review150813 --- Ship it! Ship It! - Stephan Erb On Sept. 29, 2016, 12:51

Re: Review Request 52367: Upgrade to latest pants 1.2.0 pre-release.

2016-09-28 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52367/#review150783 --- Ship it! Ship It! - Zameer Manji On Sept. 28, 2016, 3:51

Re: Review Request 52367: Upgrade to latest pants 1.2.0 pre-release.

2016-09-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52367/#review150781 --- Ship it! Master (1c1c0a5) is green with this patch.

Review Request 52367: Upgrade to latest pants 1.2.0 pre-release.

2016-09-28 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52367/ --- Review request for Aurora, Joshua Cohen and Stephan Erb. Bugs: AURORA-1784

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-28 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/#review150769 --- I tried to commit this, but e2e tests hung for me. Kai, can you

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/#review150764 --- Ship it! Master (1c1c0a5) is green with this patch.

Re: Review Request 51876: Modify executor state transition logic to rely on health checks (if enabled)

2016-09-28 Thread Kai Huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51876/ --- (Updated Sept. 28, 2016, 9:07 p.m.) Review request for Aurora, Joshua Cohen

Re: Review Request 52094: Add min_consecutive_health_checks in HealthCheckConfig

2016-09-28 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52094/#review150759 --- Ship it! Ship It! - Joshua Cohen On Sept. 28, 2016, 5 p.m.,

Re: Review Request 52343: Add Criteo to Aurora user list

2016-09-28 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52343/#review150752 --- Ship it! Ship It! - Zameer Manji On Sept. 28, 2016, 4:46

Re: Review Request 52343: Add Criteo to Aurora user list

2016-09-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52343/#review150710 --- Ship it! Master (69cba78) is green with this patch.

Review Request 52343: Add Criteo to Aurora user list

2016-09-28 Thread Pierre Cheynier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52343/ --- Review request for Aurora. Repository: aurora Description --- Add