Re: Review Request 33705: Don't retry API requests that fail with auth errors.

2015-05-04 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33705/#review82405 --- Ship it! Ship It! - Kevin Sweeney On April 30, 2015, 5:12 p.m.,

Re: Review Request 33705: Don't retry API requests that fail with auth errors.

2015-04-30 Thread Bill Farner
On April 30, 2015, 7:15 p.m., Joshua Cohen wrote: src/main/python/apache/aurora/common/transport.py, line 64 https://reviews.apache.org/r/33705/diff/1/?file=945910#file945910line64 Is this meant to indicate that it should be a function that returns a Session? Is this a standard

Re: Review Request 33705: Don't retry API requests that fail with auth errors.

2015-04-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33705/#review82071 --- Ship it! Master (f77daf7) is green with this patch.