GumpacG opened a new pull request, #3422:
URL: https://github.com/apache/tinkerpop/pull/3422

   ### Description
   The streaming-support change in #3396 regressed the python CI job from ~5-6 
min to 10-19 min and started hitting the 20-min timeout. The root cause is that 
AiohttpSyncStream.read(n) was running a full asyncio event-loop turn 
(loop.run_until_complete(...)) for every byte-level read the GraphBinary 
deserializer performs. A typical response of N results triggers at minimum 3N + 
8 reads - for a 5,000-result response that's ~500 ms of pure event-loop 
overhead vs ~0.5 ms before streaming.
   
   This PR keeps the streaming design intent but pays the asyncio cost at chunk 
granularity (~64 KB) instead of per byte.
   
   
   Assisted-by: Devin: Claude Opus 4.7


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to