---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/#review188377
---
Due to the lack of strong opinions - I'm going to ship this
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/#review188201
---
Ship it!
LGTM. I am +1 to Josh's suggestion if the additional
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/#review188181
---
Ship it!
LGTM
- Kai Huang
On Oct. 13, 2017, 12:10 a.m.,
> On Oct. 13, 2017, 2:48 p.m., Joshua Cohen wrote:
> > When I implemented this originally I wanted it to be a separate URL
> > (http://aurora/scheduler/role/env/job/completed) rather than a query
> > parameter (?tab=completed), but due to something in Angular's router it
> > wasn't really
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/#review187962
---
When I implemented this originally I wanted it to be a separate
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/#review187914
---
Master (2aee90d) is red with this patch.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62958/
---
Review request for Aurora, Kai Huang, Reza Motamedi, and Santhosh Kumar