Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/#review57416 --- Ship it! Ship It! - Joshua Cohen On Oct. 20, 2014, 7:36 p.m., Ma

Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/#review57413 --- Ship it! Ship It! - Bill Farner On Oct. 20, 2014, 7:36 p.m., Mar

Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/ --- (Updated Oct. 20, 2014, 3:36 p.m.) Review request for Aurora, Bill Farner and Z

Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Mark Chu-Carroll
> On Oct. 20, 2014, 1:17 p.m., Bill Farner wrote: > > src/main/python/apache/aurora/client/cli/cron.py, line 48 > > > > > > I don't think the "reported by server" part is a useful detail, > > consider dropping. Ditto

Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/#review57399 --- Ship it! Ship It! - Zameer Manji On Oct. 17, 2014, 10:58 a.m., M

Re: Review Request 26881: Improve error messages in client commands.

2014-10-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/#review57361 --- Ship it! src/main/python/apache/aurora/client/cli/cron.py

Re: Review Request 26881: Improve error messages in client commands.

2014-10-17 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26881/#review57206 --- Maybe this is nitpicking, but given the confusion we've already seen