You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by we...@apache.org on 2024/10/04 16:52:33 UTC
(hadoop) 02/02: HDFS-17461. Fix spotbugs in PeerCache#getInternal (#6721).
This is an automated email from the ASF dual-hosted git repository.
weichiu pushed a commit to branch branch-3.4
in repository https://gitbox.apache.org/repos/asf/hadoop.git
commit 520ded41d2baefe9d1f049db95cc2256e230f6de
Author: Sadanand Shenoy <sa...@cloudera.com>
AuthorDate: Tue Oct 1 19:41:54 2024 +0530
HDFS-17461. Fix spotbugs in PeerCache#getInternal (#6721).
---
.../src/main/java/org/apache/hadoop/hdfs/PeerCache.java | 4 ----
1 file changed, 4 deletions(-)
diff --git a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/PeerCache.java b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/PeerCache.java
index a26a518a839..41578d4d505 100644
--- a/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/PeerCache.java
+++ b/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/PeerCache.java
@@ -155,10 +155,6 @@ public class PeerCache {
private synchronized Peer getInternal(DatanodeID dnId, boolean isDomain) {
List<Value> sockStreamList = multimap.get(new Key(dnId, isDomain));
- if (sockStreamList == null) {
- return null;
- }
-
Iterator<Value> iter = sockStreamList.iterator();
while (iter.hasNext()) {
Value candidate = iter.next();
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org