Github user pnowojski commented on a diff in the pull request:
https://github.com/apache/flink/pull/6149#discussion_r195657899
--- Diff:
flink-core/src/main/java/org/apache/flink/core/fs/LimitedConnectionsFileSystem.java
---
@@ -940,7 +932,9 @@ public int read() throws IOException {
public int read(byte[] b) throws IOException {
try {
int len = originalStream.read(b);
- fs.inputRateLimiter.ifPresent(limiter ->
limiter.acquire(len));
+ if (len >= 0) {
--- End diff --
was adding this `if (len >= 0)` check a bug fix?
---
