apple / swift-nio-ssl

TLS Support for SwiftNIO, based on BoringSSL.
https://swiftpackageindex.com/apple/swift-nio-ssl/main/documentation/niossl
Apache License 2.0
388 stars 139 forks source link

Remove giant write test #398

Closed Lukasa closed 2 years ago

Lukasa commented 2 years ago

Motivation:

The giant write test requires an enormous commitment of RAM to run it. This has caused problems for both our own CI and a number of downstream CI systems, leading to long execution times or outright failures.

Unfortunately, we can't defend against the original failure using this test suite, but we can at least exercise the new machinery we added and trust that the configuration is right.

Modifications:

Result:

We feel confident that the write splitting continues to work correctly.

Resolves #388.