> On Feb. 14, 2017, 3:48 a.m., Mehrdad Nurolahzade wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java, > > line 1135 > > <https://reviews.apache.org/r/56629/diff/2/?file=1632701#file1632701line1135> > > > > We are skipping task delete through `StateManager` which publishes > > `PubsubEvent.TasksDeleted` to subscribers like `TaskGroups` and > > `JobUpdateController`. > > > > In the case of `JobUpdateController` the dependency does not seem to > > matter. Did you verify ignoring the `TaskGroups` dependency is safe?
Looks like I need to go through StateManager for TaskVars. The other subscribers only deal with PENDING tasks. It's a shame because the pubsub path is really ineffecient. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56629/#review165434 ----------------------------------------------------------- On Feb. 14, 2017, 4:12 a.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56629/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2017, 4:12 a.m.) > > > Review request for Aurora, Santhosh Kumar Shanmugham, Stephan Erb, and Zameer > Manji. > > > Repository: aurora > > > Description > ------- > > Expose task pruning endpoint in aurora_admin. Useful for scale testing in > order to 'clean up' after a test run, but also useful in production if you > have a bad actor inflating the size of your task index. > > > Diffs > ----- > > api/src/main/thrift/org/apache/aurora/gen/api.thrift > 6205c2ed1e2d5898a322de5bc2bcba6b2c0cd4b1 > > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java > a211483caf59755b1f9b286b58f183e00db6eee6 > src/main/python/apache/aurora/admin/admin.py > 070c348d2ca5db1edecf832efd9aa5481bddaa4b > src/main/python/apache/aurora/client/api/__init__.py > 1250ccd16f906eec08c159df3300d1b94b306d8e > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > 0cdd9829417bb3ef0215278a9458a3dd78a49a20 > src/test/python/apache/aurora/admin/test_admin.py > 66abade378d3974dbf031a7bbed9cd6fc4f22d5f > src/test/python/apache/aurora/client/api/test_scheduler_client.py > fab97986dcecf7761e161f4074d5108a8c362e6e > > Diff: https://reviews.apache.org/r/56629/diff/ > > > Testing > ------- > > Ran various combinations of task pruning via aurora_admin in Vagrant. Works > as expected. > > Also: > > ./pants test src/test/python/apache/aurora/admin:: > > > Thanks, > > David McLaughlin > >
