You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2014/02/13 05:41:21 UTC
svn commit: r1567847 -
/hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java
Author: tedyu
Date: Thu Feb 13 04:41:20 2014
New Revision: 1567847
URL: http://svn.apache.org/r1567847
Log:
HBASE-10510 Addendum removes HTable and uses tableName directly.
Modified:
hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java
Modified: hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java?rev=1567847&r1=1567846&r2=1567847&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java (original)
+++ hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/LoadTestTool.java Thu Feb 13 04:41:20 2014
@@ -473,20 +473,17 @@ public class LoadTestTool extends Abstra
conf.set("hadoop.security.authorization", "false");
conf.set("hadoop.security.authentication", "simple");
LOG.info("Granting permission for the user " + userOwner.getShortName());
- HTable table = new HTable(conf, tableName);
AccessControlProtos.Permission.Action[] actions = {
AccessControlProtos.Permission.Action.ADMIN,
AccessControlProtos.Permission.Action.CREATE, AccessControlProtos.Permission.Action.READ,
AccessControlProtos.Permission.Action.WRITE };
try {
- AccessControlClient.grant(conf, table.getName(), userOwner.getShortName(), COLUMN_FAMILY,
+ AccessControlClient.grant(conf, tableName, userOwner.getShortName(), COLUMN_FAMILY,
null, actions);
} catch (Throwable e) {
LOG.fatal("Error in granting permission for the user " + userOwner.getShortName(), e);
return EXIT_FAILURE;
- } finally {
- table.close();
}
}