From ec800d8b92aae791f00dfb919587fa71d99e5430 Mon Sep 17 00:00:00 2001
From: "codeflash-ai-dev[bot]"
 <157075493+codeflash-ai-dev[bot]@users.noreply.github.com>
Date: Sat, 8 Nov 2025 12:01:17 +0000
Subject: [PATCH] Optimize fetch_all_users
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The optimization replaces sequential async execution with concurrent execution using `asyncio.gather()`.

**Key Change:**
- **Original**: Fetches users one-by-one in a loop with `await fetch_user(user_id)`, blocking until each completes
- **Optimized**: Uses `asyncio.gather(*(fetch_user(user_id) for user_id in user_ids))` to launch all fetch operations concurrently

**Why This Creates a Speedup:**
The original code suffers from "false serialization" - it waits for each 0.0001-second database call to complete before starting the next one. With N users, total time is roughly N × 0.0001 seconds. The optimized version launches all fetch operations simultaneously, so total time becomes approximately max(fetch_times) ≈ 0.0001 seconds regardless of list size.

**Performance Impact:**
- **Runtime improvement**: 785% speedup (302ms → 34.1ms)
- **Throughput improvement**: 558% increase (1,862 → 12,250 operations/second)

The line profiler shows the optimization eliminates the expensive sequential loop overhead - the original `fetch_all_users` spent 96.3% of time waiting on individual fetch calls, while the optimized version completes in a single concurrent operation.

**Test Case Performance:**
The optimization excels with larger datasets - test cases with 100-500 users show the most dramatic improvements since they maximize the concurrency benefit. Small lists (1-10 users) still benefit but see smaller gains due to the fixed asyncio.sleep overhead.

This pattern is particularly valuable for I/O-bound operations like database queries, API calls, or file operations where the underlying operations can run independently.
---
 src/asynchrony/various.py | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/asynchrony/various.py b/src/asynchrony/various.py
index e5f89d2..9afc206 100644
--- a/src/asynchrony/various.py
+++ b/src/asynchrony/various.py
@@ -23,8 +23,4 @@ async def fetch_user(user_id: int) -> dict:
 
 
 async def fetch_all_users(user_ids: list[int]) -> list[dict]:
-    users = []
-    for user_id in user_ids:
-        user = await fetch_user(user_id)
-        users.append(user)
-    return users
+    return await asyncio.gather(*(fetch_user(user_id) for user_id in user_ids))