FAQ
Author: vgumashta
Date: Thu Jan 15 19:03:12 2015
New Revision: 1652225

URL: http://svn.apache.org/r1652225
Log:
HIVE-9234: HiveServer2 leaks FileSystem objects in FileSystem.CACHE (Vaibhav Gumashta reviewed by Brock Noland, Thejas Nair)

Modified:
     hive/branches/branch-0.14/service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java

Modified: hive/branches/branch-0.14/service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.14/service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java?rev=1652225&r1=1652224&r2=1652225&view=diff
==============================================================================
--- hive/branches/branch-0.14/service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java (original)
+++ hive/branches/branch-0.14/service/src/java/org/apache/hive/service/cli/session/HiveSessionImplwithUGI.java Thu Jan 15 19:03:12 2015
@@ -91,18 +91,20 @@ public class HiveSessionImplwithUGI exte
    }

    /**
- * close the file systems for the session
- * cancel the session's delegation token and close the metastore connection
+ * Close the file systems for the session and remove it from the FileSystem cache.
+ * Cancel the session's delegation token and close the metastore connection
     */
    @Override
    public void close() throws HiveSQLException {
      try {
- acquire(true);
- ShimLoader.getHadoopShims().closeAllForUGI(sessionUgi);
- cancelDelegationToken();
+ acquire(true);
+ cancelDelegationToken();
      } finally {
- release(true);
- super.close();
+ try {
+ super.close();
+ } finally {
+ ShimLoader.getHadoopShims().closeAllForUGI(sessionUgi);
+ }
      }
    }

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 1 of 1 | next ›
Discussion Overview
groupcommits @
categorieshive, hadoop
postedJan 15, '15 at 7:03p
activeJan 15, '15 at 7:03p
posts1
users1
websitehive.apache.org

1 user in discussion

Vgumashta: 1 post

People

Translate

site design / logo © 2021 Grokbase