> On March 1, 2014, 1:32 a.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/base/JobKeys.java, line 178 > > <https://reviews.apache.org/r/18655/diff/1/?file=507499#file507499line178> > > > > "from" is currently intended to unambiguously give a single IJobKey > > instance derived from role/env/name. I am not sure exposing a Set<IJobKey> > > would be a good idea. > >
It gets weird though if i call from() with a query that has a singleton set of job keys and get back Optional.absent(). - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18655/#review35889 ----------------------------------------------------------- On March 1, 2014, 12:54 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18655/ > ----------------------------------------------------------- > > (Updated March 1, 2014, 12:54 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-235 > https://issues.apache.org/jira/browse/AURORA-235 > > > Repository: aurora > > > Description > ------- > > Added an optional JobKey set filter into the TaskQuery. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/base/JobKeys.java > 5f684bc1f19470a8df4df167aa5c15ad6b9c9f95 > src/main/java/org/apache/aurora/scheduler/base/Query.java > d6f27fd6b8029401c918f942253beb59b6a71ddf > src/main/java/org/apache/aurora/scheduler/storage/mem/MemTaskStore.java > 421b33023e3b443d579685f434a4e09957b1c6e0 > src/main/thrift/org/apache/aurora/gen/api.thrift > f9fc6bcb7fdc2380043cf673a4002886192c20c1 > src/test/java/org/apache/aurora/scheduler/storage/mem/MemTaskStoreTest.java > 0c1b144bf22f1bf400da04bd5826a04891dfada2 > src/test/resources/org/apache/aurora/gen/api.thrift.md5 > 08c4c2a4717a1c827f4cef6405d78e7db5046a00 > > Diff: https://reviews.apache.org/r/18655/diff/ > > > Testing > ------- > > /build-support/jenkins/build.sh > > > Thanks, > > Maxim Khutornenko > >