You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by vi...@apache.org on 2019/04/03 04:31:46 UTC

[incubator-hudi] branch master updated: Follow up HUDI-27 : Call super.close() in HoodieWraperFileSystem::close()

This is an automated email from the ASF dual-hosted git repository.

vinoth pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git


The following commit(s) were added to refs/heads/master by this push:
     new 51f4908  Follow up HUDI-27 : Call super.close() in HoodieWraperFileSystem::close()
51f4908 is described below

commit 51f490898913db5964c208c57ab14794b42fa44d
Author: Vinoth Chandar <vi...@uber.com>
AuthorDate: Fri Mar 29 22:05:03 2019 -0700

    Follow up HUDI-27 : Call super.close() in HoodieWraperFileSystem::close()
---
 .../java/com/uber/hoodie/io/storage/HoodieWrapperFileSystem.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/hoodie-client/src/main/java/com/uber/hoodie/io/storage/HoodieWrapperFileSystem.java b/hoodie-client/src/main/java/com/uber/hoodie/io/storage/HoodieWrapperFileSystem.java
index fce9b7c..f18901d 100644
--- a/hoodie-client/src/main/java/com/uber/hoodie/io/storage/HoodieWrapperFileSystem.java
+++ b/hoodie-client/src/main/java/com/uber/hoodie/io/storage/HoodieWrapperFileSystem.java
@@ -536,9 +536,10 @@ public class HoodieWrapperFileSystem extends FileSystem {
   }
 
   @Override
-  public void close() {
-    // Don't close the underlying `fileSystem` object. This will end up closing it for every thread since it
+  public void close() throws IOException {
+    // Don't close the wrapped `fileSystem` object. This will end up closing it for every thread since it
     // could be cached across jvm. We don't own that object anyway.
+    super.close();
   }
 
   @Override