accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [13/26] git commit: Change keySet iterator to an entrySet iterator to avoid an extra get call on the Map
Date Sat, 01 Feb 2014 03:23:12 GMT
Change keySet iterator to an entrySet iterator to avoid an extra get call on the Map


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/7d9a96ab
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/7d9a96ab
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/7d9a96ab

Branch: refs/heads/2292-findbugs
Commit: 7d9a96ab3523cbdc9a7b74523890cf02919bc147
Parents: d3b19c1
Author: Josh Elser <elserj@apache.org>
Authored: Thu Jan 30 16:36:43 2014 -0500
Committer: Josh Elser <elserj@apache.org>
Committed: Fri Jan 31 22:18:52 2014 -0500

----------------------------------------------------------------------
 .../accumulo/core/client/admin/TableOperationsImpl.java      | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7d9a96ab/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
b/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
index 3f5ac8c..307243d 100644
--- a/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
+++ b/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
@@ -1236,13 +1236,13 @@ public class TableOperationsImpl extends TableOperationsHelper {
 
     try {
       FileSystem fs = FileUtil.getFileSystem(CachedConfiguration.getInstance(), instance.getConfiguration());
-      ;
+
       Map<String,String> props = getExportedProps(fs, new Path(importDir, Constants.EXPORT_FILE));
 
-      for (String propKey : props.keySet()) {
-        if (Property.isClassProperty(propKey) && !props.get(propKey).contains(Constants.CORE_PACKAGE_NAME))
{
+      for (Entry<String,String> prop : props.entrySet()) {
+        if (Property.isClassProperty(prop.getKey()) && !prop.getValue().contains(Constants.CORE_PACKAGE_NAME))
{
           Logger.getLogger(this.getClass()).info(
-              "Imported table sets '" + propKey + "' to '" + props.get(propKey) + "'.  Ensure
this class is on Accumulo classpath.");
+              "Imported table sets '" + prop.getKey() + "' to '" + prop.getValue() + "'.
 Ensure this class is on Accumulo classpath.");
         }
       }
 


Mime
View raw message