DanielLeens commented on PR #12438:
URL: https://github.com/apache/seatunnel/pull/12438#issuecomment-5855058574

   Thanks for coming back to correct this rather than leaving the earlier 
pointer standing — that is exactly the right instinct, and the 50-run 
comparison table makes the distinction clear: #12381 widens a wait that was 
never the actual bottleneck, while #12444 removes the real race by releasing 
the latch before the body write. That matches what I traced independently when 
I looked at `PayPalClient.execute()` and the test server's 
`arrived.countDown()` ordering.
   
   This PR's own diff is still just two comment corrections with no code path 
to `connector-http-paypal`, so my APPROVED review stands — the remaining red 
`Build` leg is confirmed unrelated to this change and should clear once #12444 
merges.


-- 
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