You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ec...@apache.org on 2013/11/26 23:28:57 UTC
svn commit: r1545871 -
/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java
Author: eclark
Date: Tue Nov 26 22:28:57 2013
New Revision: 1545871
URL: http://svn.apache.org/r1545871
Log:
HBASE-10036 Fix Potential Resource Leak in MetaReader
Modified:
hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java
Modified: hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java?rev=1545871&r1=1545870&r2=1545871&view=diff
==============================================================================
--- hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java (original)
+++ hbase/trunk/hbase-client/src/main/java/org/apache/hadoop/hbase/catalog/MetaReader.java Tue Nov 26 22:28:57 2013
@@ -533,8 +533,9 @@ public class MetaReader {
}
scan.addFamily(HConstants.CATALOG_FAMILY);
HTable metaTable = getMetaHTable(catalogTracker);
- ResultScanner scanner = metaTable.getScanner(scan);
+ ResultScanner scanner = null;
try {
+ scanner = metaTable.getScanner(scan);
Result data;
while((data = scanner.next()) != null) {
if (data.isEmpty()) continue;
@@ -542,7 +543,7 @@ public class MetaReader {
if (!visitor.visit(data)) break;
}
} finally {
- scanner.close();
+ if (scanner != null) scanner.close();
metaTable.close();
}
return;