Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22987 )
Change subject: [util] Remove SchedulerThread code and tests ...................................................................... [util] Remove SchedulerThread code and tests This removes the SchedulerThread implementation and its associated tests. The TestSubmitAfterShutdown case has proven unstable in ARM+TSAN environments. The code was originally introduced in KUDU-3364 but has never been used outside of tests, and the original author appears to have no plans to maintain it. Removing this unused code simplifies the codebase and improves test stability. The removal was suggested by Alexey. Change-Id: I544488158db84dca580694e884b56d5c546d89c1 Reviewed-on: http://gerrit.cloudera.org:8080/22987 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/util/threadpool-test.cc M src/kudu/util/threadpool.cc M src/kudu/util/threadpool.h 3 files changed, 3 insertions(+), 396 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22987 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I544488158db84dca580694e884b56d5c546d89c1 Gerrit-Change-Number: 22987 Gerrit-PatchSet: 3 Gerrit-Owner: KeDeng <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
