----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31491/#review74366 -----------------------------------------------------------
src/main/python/apache/thermos/monitoring/garbage.py <https://reviews.apache.org/r/31491/#comment120932> You have secured all `self._state` accesses with a check that the latter is defined, all except this one. - Stephan Erb On Feb. 26, 2015, 9 p.m., Brian Wickman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31491/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2015, 9 p.m.) > > > Review request for Aurora, Joshua Cohen and Stephan Erb. > > > Bugs: AURORA-1155 > https://issues.apache.org/jira/browse/AURORA-1155 > > > Repository: aurora > > > Description > ------- > > Add basic test coverage to the TaskGarbageCollector. > > > Diffs > ----- > > src/main/python/apache/thermos/monitoring/garbage.py > 53bb0345f963ae168206f34af1d82c67427f41fe > src/test/python/apache/thermos/monitoring/BUILD > 9c267775080e0bde08ff4e9b01862c6bee187634 > src/test/python/apache/thermos/monitoring/test_garbage.py PRE-CREATION > > Diff: https://reviews.apache.org/r/31491/diff/ > > > Testing > ------- > > ./pants test src/test/python/apache/thermos/monitoring:test_garbage > > > Thanks, > > Brian Wickman > >
