Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6178#discussion_r196740578
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/HandlerRequest.java
 ---
    @@ -39,15 +41,21 @@
     public class HandlerRequest<R extends RequestBody, M extends 
MessageParameters> {
     
        private final R requestBody;
    +   private final FileUploads uploadedFiles;
    --- End diff --
    
    This comment has not been addressed. I think the `HandlerRequest` should 
not know about the `FileUploads` because it can use to delete the files.


---

Reply via email to