| David Reiss | ea2cba8 | 2009-03-30 21:35:00 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Licensed to the Apache Software Foundation (ASF) under one |
| 3 | * or more contributor license agreements. See the NOTICE file |
| 4 | * distributed with this work for additional information |
| 5 | * regarding copyright ownership. The ASF licenses this file |
| 6 | * to you under the Apache License, Version 2.0 (the |
| 7 | * "License"); you may not use this file except in compliance |
| 8 | * with the License. You may obtain a copy of the License at |
| 9 | * |
| 10 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 11 | * |
| 12 | * Unless required by applicable law or agreed to in writing, |
| 13 | * software distributed under the License is distributed on an |
| 14 | * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY |
| 15 | * KIND, either express or implied. See the License for the |
| 16 | * specific language governing permissions and limitations |
| 17 | * under the License. |
| 18 | */ |
| Mark Slee | 9f0c651 | 2007-02-28 23:58:26 +0000 | [diff] [blame] | 19 | |
| Roger Meier | ba406d3 | 2013-07-15 22:41:34 +0200 | [diff] [blame] | 20 | #include <thrift/thrift-config.h> |
| Roger Meier | 49ff8b1 | 2012-04-13 09:12:31 +0000 | [diff] [blame] | 21 | #include <thrift/concurrency/ThreadManager.h> |
| cyy | ca8af9b | 2019-01-11 22:13:12 +0800 | [diff] [blame] | 22 | #include <thrift/concurrency/ThreadFactory.h> |
| Roger Meier | 49ff8b1 | 2012-04-13 09:12:31 +0000 | [diff] [blame] | 23 | #include <thrift/concurrency/Monitor.h> |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 24 | |
| 25 | #include <assert.h> |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 26 | #include <deque> |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 27 | #include <set> |
| 28 | #include <iostream> |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 29 | #include <stdint.h> |
| Dmytro Shteflyuk | d925d34 | 2025-11-30 17:29:12 -0500 | [diff] [blame] | 30 | #include <chrono> |
| 31 | #include <thread> |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 32 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 33 | namespace apache { |
| 34 | namespace thrift { |
| 35 | namespace concurrency { |
| 36 | namespace test { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 37 | |
| T Jake Luciani | b5e6221 | 2009-01-31 22:36:20 +0000 | [diff] [blame] | 38 | using namespace apache::thrift::concurrency; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 39 | |
| cyy | 316723a | 2019-01-05 16:35:14 +0800 | [diff] [blame] | 40 | static std::deque<std::shared_ptr<Runnable> > m_expired; |
| 41 | static void expiredNotifier(std::shared_ptr<Runnable> runnable) |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 42 | { |
| 43 | m_expired.push_back(runnable); |
| 44 | } |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 45 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 46 | static void sleep_(int64_t millisec) { |
| Dmytro Shteflyuk | d925d34 | 2025-11-30 17:29:12 -0500 | [diff] [blame] | 47 | std::this_thread::sleep_for(std::chrono::milliseconds(millisec)); |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 48 | } |
| 49 | |
| 50 | class ThreadManagerTests { |
| Konrad Grochowski | 293a40e | 2014-09-04 17:28:17 +0400 | [diff] [blame] | 51 | |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 52 | public: |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 53 | class Task : public Runnable { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 54 | |
| 55 | public: |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 56 | Task(Monitor& monitor, size_t& count, int64_t timeout) |
| James E. King, III | 3620090 | 2016-10-05 14:47:18 -0400 | [diff] [blame] | 57 | : _monitor(monitor), _count(count), _timeout(timeout), _startTime(0), _endTime(0), _done(false) {} |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 58 | |
| Sebastian Zenker | 042580f | 2019-01-29 15:48:12 +0100 | [diff] [blame] | 59 | void run() override { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 60 | |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 61 | _startTime = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 62 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 63 | sleep_(_timeout); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 64 | |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 65 | _endTime = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 66 | |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 67 | _done = true; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 68 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 69 | { |
| 70 | Synchronized s(_monitor); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 71 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 72 | // std::cout << "Thread " << _count << " completed " << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 73 | |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 74 | _count--; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 75 | if (_count % 10000 == 0) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 76 | _monitor.notify(); |
| 77 | } |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 78 | } |
| 79 | } |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 80 | |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 81 | Monitor& _monitor; |
| 82 | size_t& _count; |
| Mark Slee | 9b82d27 | 2007-05-23 05:16:07 +0000 | [diff] [blame] | 83 | int64_t _timeout; |
| 84 | int64_t _startTime; |
| 85 | int64_t _endTime; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 86 | bool _done; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 87 | Monitor _sleep; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 88 | }; |
| 89 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 90 | /** |
| 91 | * Dispatch count tasks, each of which blocks for timeout milliseconds then |
| 92 | * completes. Verify that all tasks completed and that thread manager cleans |
| 93 | * up properly on delete. |
| 94 | */ |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 95 | bool loadTest(size_t count = 100, int64_t timeout = 100LL, size_t workerCount = 4) { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 96 | |
| 97 | Monitor monitor; |
| 98 | |
| 99 | size_t activeCount = count; |
| 100 | |
| Marc Slemko | 6f038a7 | 2006-08-03 18:58:09 +0000 | [diff] [blame] | 101 | shared_ptr<ThreadManager> threadManager = ThreadManager::newSimpleThreadManager(workerCount); |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 102 | |
| cyy | ca8af9b | 2019-01-11 22:13:12 +0800 | [diff] [blame] | 103 | shared_ptr<ThreadFactory> threadFactory |
| 104 | = shared_ptr<ThreadFactory>(new ThreadFactory(false)); |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 105 | |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 106 | threadManager->threadFactory(threadFactory); |
| Marc Slemko | fe5ba12e | 2006-07-20 21:16:27 +0000 | [diff] [blame] | 107 | |
| 108 | threadManager->start(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 109 | |
| Marc Slemko | 6f038a7 | 2006-08-03 18:58:09 +0000 | [diff] [blame] | 110 | std::set<shared_ptr<ThreadManagerTests::Task> > tasks; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 111 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 112 | for (size_t ix = 0; ix < count; ix++) { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 113 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 114 | tasks.insert(shared_ptr<ThreadManagerTests::Task>( |
| 115 | new ThreadManagerTests::Task(monitor, activeCount, timeout))); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 116 | } |
| 117 | |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 118 | int64_t time00 = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| Marc Slemko | fe5ba12e | 2006-07-20 21:16:27 +0000 | [diff] [blame] | 119 | |
| Sebastian Zenker | 042580f | 2019-01-29 15:48:12 +0100 | [diff] [blame] | 120 | for (auto ix = tasks.begin(); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 121 | ix != tasks.end(); |
| 122 | ix++) { |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 123 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 124 | threadManager->add(*ix); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 125 | } |
| 126 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 127 | std::cout << "\t\t\t\tloaded " << count << " tasks to execute" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 128 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 129 | { |
| 130 | Synchronized s(monitor); |
| 131 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 132 | while (activeCount > 0) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 133 | std::cout << "\t\t\t\tactiveCount = " << activeCount << '\n'; |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 134 | monitor.wait(); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 135 | } |
| 136 | } |
| 137 | |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 138 | int64_t time01 = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 139 | |
| Mark Slee | 9b82d27 | 2007-05-23 05:16:07 +0000 | [diff] [blame] | 140 | int64_t firstTime = 9223372036854775807LL; |
| 141 | int64_t lastTime = 0; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 142 | |
| 143 | double averageTime = 0; |
| Mark Slee | 9b82d27 | 2007-05-23 05:16:07 +0000 | [diff] [blame] | 144 | int64_t minTime = 9223372036854775807LL; |
| 145 | int64_t maxTime = 0; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 146 | |
| Sebastian Zenker | 042580f | 2019-01-29 15:48:12 +0100 | [diff] [blame] | 147 | for (auto ix = tasks.begin(); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 148 | ix != tasks.end(); |
| 149 | ix++) { |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 150 | |
| Marc Slemko | 6f038a7 | 2006-08-03 18:58:09 +0000 | [diff] [blame] | 151 | shared_ptr<ThreadManagerTests::Task> task = *ix; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 152 | |
| Mark Slee | 9b82d27 | 2007-05-23 05:16:07 +0000 | [diff] [blame] | 153 | int64_t delta = task->_endTime - task->_startTime; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 154 | |
| 155 | assert(delta > 0); |
| 156 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 157 | if (task->_startTime < firstTime) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 158 | firstTime = task->_startTime; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 159 | } |
| 160 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 161 | if (task->_endTime > lastTime) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 162 | lastTime = task->_endTime; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 163 | } |
| 164 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 165 | if (delta < minTime) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 166 | minTime = delta; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 167 | } |
| 168 | |
| Mark Slee | f5f2be4 | 2006-09-05 21:05:31 +0000 | [diff] [blame] | 169 | if (delta > maxTime) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 170 | maxTime = delta; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 171 | } |
| 172 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 173 | averageTime += delta; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 174 | } |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 175 | |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 176 | averageTime /= count; |
| 177 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 178 | std::cout << "\t\t\tfirst start: " << firstTime << " Last end: " << lastTime |
| 179 | << " min: " << minTime << "ms max: " << maxTime << "ms average: " << averageTime |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 180 | << "ms" << '\n'; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 181 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 182 | bool success = (time01 - time00) >= ((int64_t)count * timeout) / (int64_t)workerCount; |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 183 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 184 | std::cout << "\t\t\t" << (success ? "Success" : "Failure") |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 185 | << "! expected time: " << ((int64_t)count * timeout) / (int64_t)workerCount << "ms elapsed time: " << time01 - time00 |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 186 | << "ms" << '\n'; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 187 | |
| Marc Slemko | c778297 | 2006-07-25 02:26:35 +0000 | [diff] [blame] | 188 | return success; |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 189 | } |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 190 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 191 | class BlockTask : public Runnable { |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 192 | |
| 193 | public: |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 194 | BlockTask(Monitor& entryMonitor, Monitor& blockMonitor, bool& blocked, Monitor& doneMonitor, size_t& count) |
| 195 | : _entryMonitor(entryMonitor), _entered(false), _blockMonitor(blockMonitor), _blocked(blocked), _doneMonitor(doneMonitor), _count(count) {} |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 196 | |
| Sebastian Zenker | 042580f | 2019-01-29 15:48:12 +0100 | [diff] [blame] | 197 | void run() override { |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 198 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 199 | Synchronized s(_entryMonitor); |
| 200 | _entered = true; |
| 201 | _entryMonitor.notify(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 202 | } |
| 203 | |
| 204 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 205 | Synchronized s(_blockMonitor); |
| 206 | while (_blocked) { |
| 207 | _blockMonitor.wait(); |
| 208 | } |
| 209 | } |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 210 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 211 | { |
| 212 | Synchronized s(_doneMonitor); |
| 213 | if (--_count == 0) { |
| 214 | _doneMonitor.notify(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 215 | } |
| 216 | } |
| 217 | } |
| 218 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 219 | Monitor& _entryMonitor; |
| 220 | bool _entered; |
| 221 | Monitor& _blockMonitor; |
| 222 | bool& _blocked; |
| 223 | Monitor& _doneMonitor; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 224 | size_t& _count; |
| 225 | }; |
| 226 | |
| 227 | /** |
| 228 | * Block test. Create pendingTaskCountMax tasks. Verify that we block adding the |
| 229 | * pendingTaskCountMax + 1th task. Verify that we unblock when a task completes */ |
| 230 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 231 | bool blockTest(int64_t timeout = 100LL, size_t workerCount = 2) { |
| 232 | (void)timeout; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 233 | bool success = false; |
| 234 | |
| 235 | try { |
| 236 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 237 | Monitor entryMonitor; // not used by this test |
| 238 | Monitor blockMonitor; |
| 239 | bool blocked[] = {true, true, true}; |
| 240 | Monitor doneMonitor; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 241 | |
| 242 | size_t pendingTaskMaxCount = workerCount; |
| 243 | |
| 244 | size_t activeCounts[] = {workerCount, pendingTaskMaxCount, 1}; |
| 245 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 246 | shared_ptr<ThreadManager> threadManager |
| 247 | = ThreadManager::newSimpleThreadManager(workerCount, pendingTaskMaxCount); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 248 | |
| cyy | ca8af9b | 2019-01-11 22:13:12 +0800 | [diff] [blame] | 249 | shared_ptr<ThreadFactory> threadFactory |
| 250 | = shared_ptr<ThreadFactory>(new ThreadFactory()); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 251 | |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 252 | threadManager->threadFactory(threadFactory); |
| 253 | |
| 254 | threadManager->start(); |
| 255 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 256 | std::vector<shared_ptr<ThreadManagerTests::BlockTask> > tasks; |
| 257 | tasks.reserve(workerCount + pendingTaskMaxCount); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 258 | |
| 259 | for (size_t ix = 0; ix < workerCount; ix++) { |
| 260 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 261 | tasks.push_back(shared_ptr<ThreadManagerTests::BlockTask>( |
| 262 | new ThreadManagerTests::BlockTask(entryMonitor, blockMonitor, blocked[0], doneMonitor, activeCounts[0]))); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 263 | } |
| 264 | |
| 265 | for (size_t ix = 0; ix < pendingTaskMaxCount; ix++) { |
| 266 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 267 | tasks.push_back(shared_ptr<ThreadManagerTests::BlockTask>( |
| 268 | new ThreadManagerTests::BlockTask(entryMonitor, blockMonitor, blocked[1], doneMonitor, activeCounts[1]))); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 269 | } |
| 270 | |
| Sebastian Zenker | 042580f | 2019-01-29 15:48:12 +0100 | [diff] [blame] | 271 | for (auto ix = tasks.begin(); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 272 | ix != tasks.end(); |
| 273 | ix++) { |
| David Reiss | 96d2388 | 2007-07-26 21:10:32 +0000 | [diff] [blame] | 274 | threadManager->add(*ix); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 275 | } |
| 276 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 277 | if (!(success = (threadManager->totalTaskCount() == pendingTaskMaxCount + workerCount))) { |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 278 | throw TException("Unexpected pending task count"); |
| 279 | } |
| 280 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 281 | shared_ptr<ThreadManagerTests::BlockTask> extraTask( |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 282 | new ThreadManagerTests::BlockTask(entryMonitor, blockMonitor, blocked[2], doneMonitor, activeCounts[2])); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 283 | |
| 284 | try { |
| 285 | threadManager->add(extraTask, 1); |
| 286 | throw TException("Unexpected success adding task in excess of pending task count"); |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 287 | } catch (TooManyPendingTasksException&) { |
| David Reiss | 9fcacc8 | 2009-06-04 00:32:54 +0000 | [diff] [blame] | 288 | throw TException("Should have timed out adding task in excess of pending task count"); |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 289 | } catch (TimedOutException&) { |
| David Reiss | 9fcacc8 | 2009-06-04 00:32:54 +0000 | [diff] [blame] | 290 | // Expected result |
| 291 | } |
| 292 | |
| 293 | try { |
| 294 | threadManager->add(extraTask, -1); |
| 295 | throw TException("Unexpected success adding task in excess of pending task count"); |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 296 | } catch (TimedOutException&) { |
| David Reiss | 9fcacc8 | 2009-06-04 00:32:54 +0000 | [diff] [blame] | 297 | throw TException("Unexpected timeout adding task in excess of pending task count"); |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 298 | } catch (TooManyPendingTasksException&) { |
| David Reiss | 9fcacc8 | 2009-06-04 00:32:54 +0000 | [diff] [blame] | 299 | // Expected result |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 300 | } |
| 301 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 302 | std::cout << "\t\t\t" |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 303 | << "Pending tasks " << threadManager->pendingTaskCount() << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 304 | |
| 305 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 306 | Synchronized s(blockMonitor); |
| 307 | blocked[0] = false; |
| 308 | blockMonitor.notifyAll(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 309 | } |
| 310 | |
| 311 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 312 | Synchronized s(doneMonitor); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 313 | while (activeCounts[0] != 0) { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 314 | doneMonitor.wait(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 315 | } |
| 316 | } |
| 317 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 318 | std::cout << "\t\t\t" |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 319 | << "Pending tasks " << threadManager->pendingTaskCount() << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 320 | |
| 321 | try { |
| 322 | threadManager->add(extraTask, 1); |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 323 | } catch (TimedOutException&) { |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 324 | std::cout << "\t\t\t" |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 325 | << "add timed out unexpectedly" << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 326 | throw TException("Unexpected timeout adding task"); |
| 327 | |
| ben-craig | fae08e7 | 2015-07-15 11:34:47 -0500 | [diff] [blame] | 328 | } catch (TooManyPendingTasksException&) { |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 329 | std::cout << "\t\t\t" |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 330 | << "add encountered too many pending exepctions" << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 331 | throw TException("Unexpected timeout adding task"); |
| 332 | } |
| 333 | |
| 334 | // Wake up tasks that were pending before and wait for them to complete |
| 335 | |
| 336 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 337 | Synchronized s(blockMonitor); |
| 338 | blocked[1] = false; |
| 339 | blockMonitor.notifyAll(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 340 | } |
| 341 | |
| 342 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 343 | Synchronized s(doneMonitor); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 344 | while (activeCounts[1] != 0) { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 345 | doneMonitor.wait(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 346 | } |
| 347 | } |
| 348 | |
| 349 | // Wake up the extra task and wait for it to complete |
| 350 | |
| 351 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 352 | Synchronized s(blockMonitor); |
| 353 | blocked[2] = false; |
| 354 | blockMonitor.notifyAll(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 355 | } |
| 356 | |
| 357 | { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 358 | Synchronized s(doneMonitor); |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 359 | while (activeCounts[2] != 0) { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 360 | doneMonitor.wait(); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 361 | } |
| 362 | } |
| 363 | |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 364 | threadManager->stop(); |
| 365 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 366 | if (!(success = (threadManager->totalTaskCount() == 0))) { |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 367 | throw TException("Unexpected total task count"); |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 368 | } |
| 369 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 370 | } catch (TException& e) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 371 | std::cout << "ERROR: " << e.what() << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 372 | } |
| 373 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 374 | std::cout << "\t\t\t" << (success ? "Success" : "Failure") << '\n'; |
| Marc Slemko | 3a3b53b | 2007-05-22 23:59:54 +0000 | [diff] [blame] | 375 | return success; |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 376 | } |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 377 | |
| 378 | |
| 379 | bool apiTest() { |
| 380 | |
| 381 | // prove currentTime has milliseconds granularity since many other things depend on it |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 382 | int64_t a = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 383 | sleep_(100); |
| cyy | bfdbd03 | 2019-01-12 14:38:28 +0800 | [diff] [blame] | 384 | int64_t b = std::chrono::duration_cast<std::chrono::milliseconds>(std::chrono::steady_clock::now().time_since_epoch()).count(); |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 385 | if (b - a < 50 || b - a > 150) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 386 | std::cerr << "\t\t\texpected 100ms gap, found " << (b-a) << "ms gap instead." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 387 | return false; |
| 388 | } |
| 389 | |
| cyy | ca8af9b | 2019-01-11 22:13:12 +0800 | [diff] [blame] | 390 | return apiTestWithThreadFactory(shared_ptr<ThreadFactory>(new ThreadFactory())); |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 391 | |
| 392 | } |
| 393 | |
| cyy | ca8af9b | 2019-01-11 22:13:12 +0800 | [diff] [blame] | 394 | bool apiTestWithThreadFactory(shared_ptr<ThreadFactory> threadFactory) |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 395 | { |
| 396 | shared_ptr<ThreadManager> threadManager = ThreadManager::newSimpleThreadManager(1); |
| 397 | threadManager->threadFactory(threadFactory); |
| 398 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 399 | std::cout << "\t\t\t\tstarting.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 400 | |
| 401 | threadManager->start(); |
| cyy | 316723a | 2019-01-05 16:35:14 +0800 | [diff] [blame] | 402 | threadManager->setExpireCallback(expiredNotifier); // std::bind(&ThreadManagerTests::expiredNotifier, this)); |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 403 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 404 | #define EXPECT(FUNC, COUNT) { size_t c = FUNC; if (c != COUNT) { std::cerr << "expected " #FUNC" to be " #COUNT ", but was " << c << '\n'; return false; } } |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 405 | |
| 406 | EXPECT(threadManager->workerCount(), 1); |
| 407 | EXPECT(threadManager->idleWorkerCount(), 1); |
| 408 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 409 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 410 | std::cout << "\t\t\t\tadd 2nd worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 411 | |
| 412 | threadManager->addWorker(); |
| 413 | |
| 414 | EXPECT(threadManager->workerCount(), 2); |
| 415 | EXPECT(threadManager->idleWorkerCount(), 2); |
| 416 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 417 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 418 | std::cout << "\t\t\t\tremove 2nd worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 419 | |
| 420 | threadManager->removeWorker(); |
| 421 | |
| 422 | EXPECT(threadManager->workerCount(), 1); |
| 423 | EXPECT(threadManager->idleWorkerCount(), 1); |
| 424 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 425 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 426 | std::cout << "\t\t\t\tremove 1st worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 427 | |
| 428 | threadManager->removeWorker(); |
| 429 | |
| 430 | EXPECT(threadManager->workerCount(), 0); |
| 431 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 432 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 433 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 434 | std::cout << "\t\t\t\tadd blocking task.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 435 | |
| 436 | // We're going to throw a blocking task into the mix |
| 437 | Monitor entryMonitor; // signaled when task is running |
| 438 | Monitor blockMonitor; // to be signaled to unblock the task |
| 439 | bool blocked(true); // set to false before notifying |
| 440 | Monitor doneMonitor; // signaled when count reaches zero |
| 441 | size_t activeCount = 1; |
| 442 | shared_ptr<ThreadManagerTests::BlockTask> blockingTask( |
| 443 | new ThreadManagerTests::BlockTask(entryMonitor, blockMonitor, blocked, doneMonitor, activeCount)); |
| 444 | threadManager->add(blockingTask); |
| 445 | |
| 446 | EXPECT(threadManager->workerCount(), 0); |
| 447 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 448 | EXPECT(threadManager->pendingTaskCount(), 1); |
| 449 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 450 | std::cout << "\t\t\t\tadd other task.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 451 | |
| 452 | shared_ptr<ThreadManagerTests::Task> otherTask( |
| 453 | new ThreadManagerTests::Task(doneMonitor, activeCount, 0)); |
| 454 | |
| 455 | threadManager->add(otherTask); |
| 456 | |
| 457 | EXPECT(threadManager->workerCount(), 0); |
| 458 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 459 | EXPECT(threadManager->pendingTaskCount(), 2); |
| 460 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 461 | std::cout << "\t\t\t\tremove blocking task specifically.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 462 | |
| 463 | threadManager->remove(blockingTask); |
| 464 | |
| 465 | EXPECT(threadManager->workerCount(), 0); |
| 466 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 467 | EXPECT(threadManager->pendingTaskCount(), 1); |
| 468 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 469 | std::cout << "\t\t\t\tremove next pending task.." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 470 | |
| 471 | shared_ptr<Runnable> nextTask = threadManager->removeNextPending(); |
| 472 | if (nextTask != otherTask) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 473 | std::cerr << "\t\t\t\t\texpected removeNextPending to return otherTask" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 474 | return false; |
| 475 | } |
| 476 | |
| 477 | EXPECT(threadManager->workerCount(), 0); |
| 478 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 479 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 480 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 481 | std::cout << "\t\t\t\tremove next pending task (none left).." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 482 | |
| 483 | nextTask = threadManager->removeNextPending(); |
| 484 | if (nextTask) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 485 | std::cerr << "\t\t\t\t\texpected removeNextPending to return an empty Runnable" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 486 | return false; |
| 487 | } |
| 488 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 489 | std::cout << "\t\t\t\tadd 2 expired tasks and 1 not.." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 490 | |
| 491 | shared_ptr<ThreadManagerTests::Task> expiredTask( |
| 492 | new ThreadManagerTests::Task(doneMonitor, activeCount, 0)); |
| 493 | |
| 494 | threadManager->add(expiredTask, 0, 1); |
| 495 | threadManager->add(blockingTask); // add one that hasn't expired to make sure it gets skipped |
| 496 | threadManager->add(expiredTask, 0, 1); // add a second expired to ensure removeExpiredTasks removes both |
| 497 | |
| 498 | sleep_(50); // make sure enough time elapses for it to expire - the shortest expiration time is 1 millisecond |
| 499 | |
| 500 | EXPECT(threadManager->workerCount(), 0); |
| 501 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 502 | EXPECT(threadManager->pendingTaskCount(), 3); |
| 503 | EXPECT(threadManager->expiredTaskCount(), 0); |
| 504 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 505 | std::cout << "\t\t\t\tremove expired tasks.." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 506 | |
| 507 | if (!m_expired.empty()) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 508 | std::cerr << "\t\t\t\t\texpected m_expired to be empty" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 509 | return false; |
| 510 | } |
| 511 | |
| 512 | threadManager->removeExpiredTasks(); |
| 513 | |
| 514 | if (m_expired.size() != 2) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 515 | std::cerr << "\t\t\t\t\texpected m_expired to be set" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 516 | return false; |
| 517 | } |
| 518 | |
| 519 | if (m_expired.front() != expiredTask) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 520 | std::cerr << "\t\t\t\t\texpected m_expired[0] to be the expired task" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 521 | return false; |
| 522 | } |
| 523 | m_expired.pop_front(); |
| 524 | |
| 525 | if (m_expired.front() != expiredTask) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 526 | std::cerr << "\t\t\t\t\texpected m_expired[1] to be the expired task" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 527 | return false; |
| 528 | } |
| 529 | |
| 530 | m_expired.clear(); |
| 531 | |
| 532 | threadManager->remove(blockingTask); |
| 533 | |
| 534 | EXPECT(threadManager->workerCount(), 0); |
| 535 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 536 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 537 | EXPECT(threadManager->expiredTaskCount(), 2); |
| 538 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 539 | std::cout << "\t\t\t\tadd expired task (again).." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 540 | |
| 541 | threadManager->add(expiredTask, 0, 1); // expires in 1ms |
| 542 | sleep_(50); // make sure enough time elapses for it to expire - the shortest expiration time is 1ms |
| 543 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 544 | std::cout << "\t\t\t\tadd worker to consume expired task.." << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 545 | |
| 546 | threadManager->addWorker(); |
| 547 | sleep_(100); // make sure it has time to spin up and expire the task |
| 548 | |
| 549 | if (m_expired.empty()) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 550 | std::cerr << "\t\t\t\t\texpected m_expired to be set" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 551 | return false; |
| 552 | } |
| 553 | |
| 554 | if (m_expired.front() != expiredTask) { |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 555 | std::cerr << "\t\t\t\t\texpected m_expired to be the expired task" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 556 | return false; |
| 557 | } |
| 558 | |
| 559 | m_expired.clear(); |
| 560 | |
| 561 | EXPECT(threadManager->workerCount(), 1); |
| 562 | EXPECT(threadManager->idleWorkerCount(), 1); |
| 563 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 564 | EXPECT(threadManager->expiredTaskCount(), 3); |
| 565 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 566 | std::cout << "\t\t\t\ttry to remove too many workers" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 567 | try { |
| 568 | threadManager->removeWorker(2); |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 569 | std::cerr << "\t\t\t\t\texpected InvalidArgumentException" << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 570 | return false; |
| 571 | } catch (const InvalidArgumentException&) { |
| 572 | /* expected */ |
| 573 | } |
| 574 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 575 | std::cout << "\t\t\t\tremove worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 576 | |
| 577 | threadManager->removeWorker(); |
| 578 | |
| 579 | EXPECT(threadManager->workerCount(), 0); |
| 580 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 581 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 582 | EXPECT(threadManager->expiredTaskCount(), 3); |
| 583 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 584 | std::cout << "\t\t\t\tadd blocking task.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 585 | |
| 586 | threadManager->add(blockingTask); |
| 587 | |
| 588 | EXPECT(threadManager->workerCount(), 0); |
| 589 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 590 | EXPECT(threadManager->pendingTaskCount(), 1); |
| 591 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 592 | std::cout << "\t\t\t\tadd worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 593 | |
| 594 | threadManager->addWorker(); |
| 595 | { |
| 596 | Synchronized s(entryMonitor); |
| 597 | while (!blockingTask->_entered) { |
| 598 | entryMonitor.wait(); |
| 599 | } |
| 600 | } |
| 601 | |
| 602 | EXPECT(threadManager->workerCount(), 1); |
| 603 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 604 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 605 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 606 | std::cout << "\t\t\t\tunblock task and remove worker.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 607 | |
| 608 | { |
| 609 | Synchronized s(blockMonitor); |
| 610 | blocked = false; |
| 611 | blockMonitor.notifyAll(); |
| 612 | } |
| 613 | threadManager->removeWorker(); |
| 614 | |
| 615 | EXPECT(threadManager->workerCount(), 0); |
| 616 | EXPECT(threadManager->idleWorkerCount(), 0); |
| 617 | EXPECT(threadManager->pendingTaskCount(), 0); |
| 618 | |
| CJCombrink | 4a280d5 | 2024-03-14 19:57:41 +0100 | [diff] [blame] | 619 | std::cout << "\t\t\t\tcleanup.. " << '\n'; |
| James E. King, III | df89913 | 2016-11-12 15:16:30 -0500 | [diff] [blame] | 620 | |
| 621 | blockingTask.reset(); |
| 622 | threadManager.reset(); |
| 623 | return true; |
| 624 | } |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 625 | }; |
| Marc Slemko | 6f038a7 | 2006-08-03 18:58:09 +0000 | [diff] [blame] | 626 | |
| Konrad Grochowski | 16a23a6 | 2014-11-13 15:33:38 +0100 | [diff] [blame] | 627 | } |
| 628 | } |
| 629 | } |
| 630 | } // apache::thrift::concurrency |
| Marc Slemko | 740343d | 2006-07-20 00:31:02 +0000 | [diff] [blame] | 631 | |
| T Jake Luciani | b5e6221 | 2009-01-31 22:36:20 +0000 | [diff] [blame] | 632 | using namespace apache::thrift::concurrency::test; |