feat: support concurrent chunk uploads#95
Conversation
Greptile SummaryThis PR refactors chunked file uploads to run up to 8 chunks concurrently using
Confidence Score: 5/5Safe to merge; the concurrency logic is well-structured and the header-isolation fix removes a latent cross-request mutation bug. The first chunk is uploaded sequentially before workers start, so uploadId is written exactly once before any parallel reads. Worker coordination via Interlocked and resultLock is correct. The only concern is a synchronous lock for stream reads that blocks thread pool threads, but this is a design-quality issue rather than a correctness defect. Appwrite/Client.cs — the stream-read locking pattern in ReadChunkAsync is worth revisiting for thread-pool-constrained hosts. Important Files Changed
Reviews (2): Last reviewed commit: "feat: support concurrent chunk uploads" | Re-trigger Greptile |
This PR updates the SDK to support concurrent chunk uploads.