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

Hudson commented on HDFS-14923:
-------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #17576 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/17576/])
HDFS-14923. Remove dead code from HealthMonitor. Contributed by Fei Hui. 
(weichiu: rev 7be5508d9b35892f483ba6022b6aced7648b8fa3)
* (edit) 
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ha/HealthMonitor.java


> Remove dead code from HealthMonitor
> -----------------------------------
>
>                 Key: HDFS-14923
>                 URL: https://issues.apache.org/jira/browse/HDFS-14923
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>    Affects Versions: 3.3.0, 3.2.1, 3.1.3
>            Reporter: Fei Hui
>            Assignee: Fei Hui
>            Priority: Minor
>             Fix For: 3.3.0, 3.1.4, 3.2.2
>
>         Attachments: HDFS-14923.001.patch
>
>
> Dig ZKFC source code and find that the dead code as follow
> {code}
> public void removeCallback(Callback cb) {
>    callbacks.remove(cb);
> }
> public synchronized void removeServiceStateCallback(ServiceStateCallback cb) {
>    serviceStateCallbacks.remove(cb);
> }
> synchronized HAServiceStatus getLastServiceStatus() {
>    return lastServiceState;
> }
> {code}
> It's useless, and should be deleted.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to