----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27317/#review58937 -----------------------------------------------------------
Ship it! src/main/java/org/apache/aurora/scheduler/quota/QuotaInfo.java <https://reviews.apache.org/r/27317/#comment100124> java.util.Objects src/main/java/org/apache/aurora/scheduler/quota/QuotaInfo.java <https://reviews.apache.org/r/27317/#comment100123> You'll need to fully-qualify this until we're on Guava 18, which renames this class to MoreObjects. ```java return com.google.common.base.Objects.toStringHelper(this) ... .toString(); ``` - Kevin Sweeney On Oct. 30, 2014, 10:11 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27317/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2014, 10:11 a.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-825 > https://issues.apache.org/jira/browse/AURORA-825 > > > Repository: aurora > > > Description > ------- > > Cron job consumption is now calculated as MAX(SUM(template), SUM(tasks)). > This is rolled up into overall role consumption for reporting and quota > checks. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/quota/QuotaInfo.java > d4e0f5335c9ebd8da97cb9634152b46c2bd9affe > src/main/java/org/apache/aurora/scheduler/quota/QuotaManager.java > 3fbab8b27403a416e86c97970eb6124c8a34c38c > src/test/java/org/apache/aurora/scheduler/quota/QuotaManagerImplTest.java > 2fe257416e8fc11be3ebf0dabbf4f63628b3c670 > > Diff: https://reviews.apache.org/r/27317/diff/ > > > Testing > ------- > > ./gradlew -Pq build > > > Thanks, > > Maxim Khutornenko > >