apple / swift-nio

Event-driven network application framework for high performance protocol servers & clients, non-blocking.
https://swiftpackageindex.com/apple/swift-nio/documentation
Apache License 2.0
7.94k stars 645 forks source link

NIOThrowingAsyncSequenceProducer gets the FileHandle tests to hang on linux/Android #2887

Open finagolfin opened 2 weeks ago

finagolfin commented 2 weeks ago

Expected behavior

The trunk tests run and pass.

Actual behavior

The FileHandle tests hang 80-90% of the time.

Steps to reproduce

  1. ../swift-6.0-RELEASE-fedora39/usr/bin/swift test --filter FileHandleTests

If possible, minimal yet complete reproducer code (or URL to code)

Reverting #2879 gets everything working well again.

SwiftNIO version/commit hash

8307ad610

System & version information

Fedora 40 x86_64 using Swift 6.0 and the Android x86_64 emulator running the trunk NIO tests compiled by 5.10.1 or the 6.0/6.1 snapshot toolchains

rnro commented 2 weeks ago

Hi! Thanks for filing this issue. I am currently taking a look into this problem to see if I can come up with a fix in the short term, however if that's not possible I will revert the change and investigate it separately. We certainly won't be cutting a release with the code in its current state.