rest-bench: drain work queue before exitingwip-3896

Fixes: #3896
We assert if workqueue is not empty on shutdown. This makes it so that
if there are still running threads, we're going to wait on them. Which
means that if there's an error in handling requests, we might wait
forever.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

1 files changed, 3 insertions, 0 deletions

diff --git a/src/tools/rest_bench.cc b/src/tools/rest_bench.cc
index feea4de4932..b6bdd2b5e18 100644
--- a/src/tools/rest_bench.cc
+++ b/src/tools/rest_bench.cc
@@ -282,6 +282,9 @@ public:
     list_bucket_handler.listBucketCallback = list_bucket_callback;
 
   }
+  ~RESTDispatcher() {
+    req_wq.stop();
+  }
   void process_context(req_context *ctx);
   void get_obj(req_context *ctx);
   void put_obj(req_context *ctx);