hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jing Zhao (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-13546) Override equals and hashCode to avoid connection leakage
Date Thu, 01 Sep 2016 17:54:20 GMT

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

Jing Zhao commented on HADOOP-13546:
------------------------------------

Comments on the latest patch:
# TestConnectionRetryPolicy.java does not have ASF license.
# The following code can be simplified as "return (obj instanceof WrapperRetryPolicy) &&
this.multipleLinearRandomRetry.equals(((WrapperRetryPolicy) obj).multipleLinearRandomRetry);"
{code}
      } else if (obj == null || obj.getClass() != this.getClass()) {
        return false;
      } else {
        return this.multipleLinearRandomRetry
            .equals(((WrapperRetryPolicy) obj).multipleLinearRandomRetry);
      }
{code}
# The new javadoc is not accurate. To ignore remoteExceptionToRetry is mainly because our
motivation for overriding equals and hashCode. The motivation here is for connection reuse,
and remoteExceptionToRetry does not affect connection failure handling.
# In {{testDefaultRetryPolicyReuseConnections}}, we can add another test case to check default
retry policy with non-null remoteExceptionToRetry case.

> Override equals and hashCode to avoid connection leakage
> --------------------------------------------------------
>
>                 Key: HADOOP-13546
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13546
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: ipc
>    Affects Versions: 2.7.0
>            Reporter: Xiaobing Zhou
>            Assignee: Xiaobing Zhou
>         Attachments: HADOOP-13546-HADOOP-13436.000.patch, HADOOP-13546-HADOOP-13436.001.patch,
HADOOP-13546-HADOOP-13436.002.patch, HADOOP-13546-HADOOP-13436.003.patch, HADOOP-13546-HADOOP-13436.004.patch,
HADOOP-13546-HADOOP-13436.005.patch, HADOOP-13546-HADOOP-13436.006.patch
>
>
> Override #equals and #hashcode to ensure multiple instances are equivalent. They eventually
> share the same RPC connection given the other arguments of constructing ConnectionId
are
> the same.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message