[ 
https://issues.apache.org/jira/browse/HDFS-13232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16387106#comment-16387106
 ] 

Íñigo Goiri commented on HDFS-13232:
------------------------------------

This together with HDFS-13230 shows a clear hole in the unit testing for the 
connection manager.

> RBF: ConnectionPool should return first usable connection
> ---------------------------------------------------------
>
>                 Key: HDFS-13232
>                 URL: https://issues.apache.org/jira/browse/HDFS-13232
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Wei Yan
>            Assignee: Wei Yan
>            Priority: Minor
>
> In current ConnectionPool.getConnection(), it will return the first active 
> connection:
> {code:java}
> for (int i=0; i<size; i++) {
>   int index = (threadIndex + i) % size;
>   conn = tmpConnections.get(index);
>   if (conn != null && !conn.isUsable()) {
>     return conn;
>   }
> }
> {code}
> Here "!conn.isUsable()" should be "conn.isUsable()".



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to