> On May 25, 2017, 4:35 p.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/http/State.java > > Lines 50 (patched) > > <https://reviews.apache.org/r/59565/diff/1/?file=1732331#file1732331line50> > > > > Call this `taskConfigLookupId` or `taskConfigId`? Calling it `taskId` > > starts colliding with the existing concept of taskIds.
Renamed to taskKey. > On May 25, 2017, 4:35 p.m., Santhosh Kumar Shanmugham wrote: > > src/main/java/org/apache/aurora/scheduler/http/State.java > > Lines 64 (patched) > > <https://reviews.apache.org/r/59565/diff/1/?file=1732331#file1732331line64> > > > > Call this `taskConfigs`? Done. > On May 25, 2017, 4:35 p.m., Santhosh Kumar Shanmugham wrote: > > src/test/java/org/apache/aurora/scheduler/http/StateTest.java > > Lines 79-81 (patched) > > <https://reviews.apache.org/r/59565/diff/1/?file=1732332#file1732332line79> > > > > Call these `taskConfigLookupId1` or `taskConfigId1` so on? Renamed to key. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59565/#review176092 ----------------------------------------------------------- On May 25, 2017, 2:43 a.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59565/ > ----------------------------------------------------------- > > (Updated May 25, 2017, 2:43 a.m.) > > > Review request for Aurora, Santhosh Kumar Shanmugham and Zameer Manji. > > > Repository: aurora > > > Description > ------- > > The state endpoint was over 400MB in our scale test environment (!!). This > shrinks it down to 14MB. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/http/State.java > dc356b77206d81bce7c8172248f08cd9fb8d7f30 > src/test/java/org/apache/aurora/scheduler/http/StateTest.java > 0af8b0d79a4d80276b4bf5bedfe26d3c076c40d8 > > > Diff: https://reviews.apache.org/r/59565/diff/1/ > > > Testing > ------- > > $ ls -al > -rw-r--r-- 1 dmclaughlin 440799949 May 24 15:15 aurora-state.json > -rw-r--r-- 1 dmclaughlin 14636611 May 24 19:35 > aurora-state-normalized.json > > > Thanks, > > David McLaughlin > >
