Re: Review Request 34570: Use set-literals and set-comprehensions where possible

2015-06-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34570/#review86883 --- Master (a1f7b3d) is red with this patch. ./build-support/jenkins/b

Re: Review Request 34570: Use set-literals and set-comprehensions where possible

2015-06-05 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34570/#review86879 --- @ReviewBot retry - Joshua Cohen On May 21, 2015, 11:28 p.m., Kevi

Re: Review Request 34570: Use set-literals and set-comprehensions where possible

2015-06-05 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34570/#review86878 --- Ship it! Presumably you'll need to rebase. - Joshua Cohen On May

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-06-05 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34661/#review86874 --- Ship it! Validated the links point to working videos or slide decks

Re: Review Request 34570: Use set-literals and set-comprehensions where possible

2015-06-05 Thread Kevin Sweeney
> On May 21, 2015, 5:22 p.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/client/api/__init__.py, line 114 > > > > > > Do we care that we're going from an immutable to a mutable set with > > this change (her

Re: Review Request 34570: Use set-literals and set-comprehensions where possible

2015-06-05 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34570/#review86872 --- Ship it! Also LGTM pending jcohen's comment - Joe Smith On May 2