This is an automated email from the ASF dual-hosted git repository.

hexiaoqiao pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 6ccb223c9c98 HDFS-17461. Fix spotbugs in PeerCache#getInternal 
(#6721). Contributed by Haiyang Hu.
6ccb223c9c98 is described below

commit 6ccb223c9c98a448bdb46acfddeb0b351d5b196a
Author: huhaiyang <huhaiyang...@126.com>
AuthorDate: Fri Apr 12 19:53:25 2024 +0800

    HDFS-17461. Fix spotbugs in PeerCache#getInternal (#6721). Contributed by 
Haiyang Hu.
    
    Reviewed-by: ZanderXu <zande...@apache.org>
    Reviewed-by: Ayush Saxena <ayushsax...@apache.org>
    Signed-off-by: He Xiaoqiao <hexiaoq...@apache.org>
---
 .../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 a26a518a8395..41578d4d505d 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-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org

Reply via email to