Re: Review Request 44801: Fix ArtifactResourceProviderTest to avoid set ordering issues

2016-03-14 Thread Robert Levas
> On March 14, 2016, 3:39 p.m., Jonathan Hurley wrote: > > I +1'd this, but could you also look at: > > > > AMBARI-14013 > > > > And "CollectionPresentationUtils" ... I think this is the same issue, no? > > Maybe you're re-inventing the wheel when you don't have to. The issue with `ExecutionC

Re: Review Request 44801: Fix ArtifactResourceProviderTest to avoid set ordering issues

2016-03-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44801/#review123492 --- I +1'd this, but could you also look at: AMBARI-14013 And "Colle

Re: Review Request 44801: Fix ArtifactResourceProviderTest to avoid set ordering issues

2016-03-14 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44801/#review123489 --- Ship it! Ship It! - Jonathan Hurley On March 14, 2016, 3:09