sorry, I did not miss it but forgot to react... There are two things that I don't like about removing this test: - how is `StreamNetworkThroughputBenchmarkTest#largeRemoteMode` a stress test for concurrently releasing and notifying about new buffers? - a benchmark should be about performance while a test should cover correctness: even if a performance test does cover correctness at the moment, that could too easily be changed in future
[ Full content available at: https://github.com/apache/flink/pull/6555 ] This message was relayed via gitbox.apache.org for [email protected]
