Commit 87d47ff
authored
Refactor(regen): drop redundant concurrency Semaphore (#718)
## Purpose
Remove the `asyncio.Semaphore` from
`scripts/response_regeneration/script.py`
Why: The semaphore was a no-op. main spawns exactly `args.concurrency`
workers, and each worker awaits one request at a time before pulling the
next item from the queue.
## Tests
Nothing broken.
## Checklist
I have filled in:
- [x] The purpose of the PR, such as "Fix some issue (link existing
issues this PR will resolve)".
- [x] The test plan/results, such as providing test command and pasting
the results.
- [ ] (Optional) The necessary documentation update.
- [x] I (a human) have written or reviewed the code in this pr to the
best of my ability.
Signed-off-by: Ranran Haoran Zhang <ranzhang@redhat.com>1 parent 74e30d3 commit 87d47ff
1 file changed
Lines changed: 2 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
186 | 186 | | |
187 | 187 | | |
188 | 188 | | |
189 | | - | |
190 | 189 | | |
191 | 190 | | |
192 | 191 | | |
| |||
197 | 196 | | |
198 | 197 | | |
199 | 198 | | |
200 | | - | |
| 199 | + | |
201 | 200 | | |
202 | 201 | | |
203 | 202 | | |
204 | 203 | | |
205 | 204 | | |
206 | 205 | | |
207 | 206 | | |
208 | | - | |
209 | 207 | | |
210 | 208 | | |
211 | 209 | | |
| |||
253 | 251 | | |
254 | 252 | | |
255 | 253 | | |
256 | | - | |
| 254 | + | |
257 | 255 | | |
258 | 256 | | |
259 | 257 | | |
| |||
363 | 361 | | |
364 | 362 | | |
365 | 363 | | |
366 | | - | |
367 | 364 | | |
368 | 365 | | |
369 | 366 | | |
| |||
391 | 388 | | |
392 | 389 | | |
393 | 390 | | |
394 | | - | |
395 | 391 | | |
396 | 392 | | |
397 | 393 | | |
| |||
0 commit comments