masteryhx commented on code in PR #20306:
URL: https://github.com/apache/flink/pull/20306#discussion_r930578280


##########
flink-dstl/flink-dstl-dfs/src/main/java/org/apache/flink/changelog/fs/StateChangeFsUploader.java:
##########
@@ -168,7 +168,7 @@ private UploadTasksResult upload(Path path, 
Collection<UploadTask> tasks) throws
                 wrappedStreamClosed = true;
             }
         } finally {
-            if (!wrappedStreamClosed) {
+            if (!wrappedStreamClosed || compression) {
                 fsStream.close();
             }

Review Comment:
   I have tried to remove and rerun it, it also cannot work. It's not the 
reason.
   Actually, I found the the case will always fail whether enabling compression 
or not.
   What confused me is that the case will always fail only if I warp the stream 
even if there is no other logic.
   And It just keep running until timeout and don't thrown an exception.



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to