> On Jan. 23, 2014, 6:17 p.m., Bill Farner wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 456 > > <https://reviews.apache.org/r/17205/diff/1/?file=435558#file435558line456> > > > > this one too
Will do this in the next diff. Closing out this diff since I am blocked on these changes. - Suman ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17205/#review32638 ----------------------------------------------------------- On Jan. 22, 2014, 9:02 p.m., Suman Karumuri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17205/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 9:02 p.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Repository: aurora > > > Description > ------- > > Moved readonly thrift calls into ReadOnlyScheduler. > > Not sure if I we need to update the release version, since the change in > theory should have no impact on the consumers of AuroraSchedulerManager > service. > > > Diffs > ----- > > src/main/thrift/org/apache/aurora/gen/api.thrift > 86220ff1ef42c892c10d4d9f174552e113a32e13 > src/test/resources/org/apache/aurora/gen/api.thrift.md5 > fd64486908839ede8a326aa0603f16ce677d7eb8 > > Diff: https://reviews.apache.org/r/17205/diff/ > > > Testing > ------- > > gradle clean build > gradle run to test with local UI. > > > Thanks, > > Suman Karumuri > >