Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-10397 Fix findbugs introduced from HBASE-9426. 

git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1560427 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
anoopsamjohn 2014-01-22 17:19:41 +00:00
parent eb998894ab
commit b5d161c513
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
View File

@ -24,11 +24,10 @@ import java.io.InterruptedIOException;
import java.net.SocketTimeoutException; import java.net.SocketTimeoutException;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Arrays; import java.util.Arrays;
import java.util.HashMap;
import java.util.LinkedList; import java.util.LinkedList;
import java.util.List; import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Properties; import java.util.Map.Entry;
import java.util.concurrent.atomic.AtomicInteger; import java.util.concurrent.atomic.AtomicInteger;
import java.util.concurrent.atomic.AtomicReference; import java.util.concurrent.atomic.AtomicReference;
import java.util.regex.Pattern; import java.util.regex.Pattern;
@ -3013,9 +3012,9 @@ public class HBaseAdmin implements Abortable, Closeable {
Map<String, String> props) throws IOException { Map<String, String> props) throws IOException {
ProcedureDescription.Builder builder = ProcedureDescription.newBuilder(); ProcedureDescription.Builder builder = ProcedureDescription.newBuilder();
builder.setSignature(signature).setInstance(instance); builder.setSignature(signature).setInstance(instance);
for (String key : props.keySet()) { for (Entry<String, String> entry : props.entrySet()) {
NameStringPair pair = NameStringPair.newBuilder().setName(key) NameStringPair pair = NameStringPair.newBuilder().setName(entry.getKey())
.setValue(props.get(key)).build(); .setValue(entry.getValue()).build();
builder.addConfiguration(pair); builder.addConfiguration(pair);
} }
@ -3081,9 +3080,9 @@ public class HBaseAdmin implements Abortable, Closeable {
throws IOException { throws IOException {
final ProcedureDescription.Builder builder = ProcedureDescription.newBuilder(); final ProcedureDescription.Builder builder = ProcedureDescription.newBuilder();
builder.setSignature(signature).setInstance(instance); builder.setSignature(signature).setInstance(instance);
for (String key : props.keySet()) { for (Entry<String, String> entry : props.entrySet()) {
NameStringPair pair = NameStringPair.newBuilder().setName(key) NameStringPair pair = NameStringPair.newBuilder().setName(entry.getKey())
.setValue(props.get(key)).build(); .setValue(entry.getValue()).build();
builder.addConfiguration(pair); builder.addConfiguration(pair);
} }
final ProcedureDescription desc = builder.build(); final ProcedureDescription desc = builder.build();