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

Yu Li commented on HBASE-17526:
-------------------------------

Observed the same issue in [pre-commit 
check|https://issues.apache.org/jira/browse/HBASE-17538?focusedCommentId=15839541&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15839541]
 for HBASE-17538, with below findbugs complaint:
{noformat}
UrF     Unread field: 
org.apache.hadoop.hbase.master.procedure.MasterProcedureScheduler$Queue.suspended

In class org.apache.hadoop.hbase.master.procedure.MasterProcedureScheduler$Queue
Field 
org.apache.hadoop.hbase.master.procedure.MasterProcedureScheduler$Queue.suspended
At MasterProcedureScheduler.java:[line 970]
{noformat}
Let me upload an addendum patch to avoid the further complaint for other 
patches.

> Procedure v2 - cleanup isSuspended from MasterProcedureScheduler#Queue
> ----------------------------------------------------------------------
>
>                 Key: HBASE-17526
>                 URL: https://issues.apache.org/jira/browse/HBASE-17526
>             Project: HBase
>          Issue Type: Sub-task
>          Components: proc-v2
>            Reporter: Appy
>            Assignee: Appy
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-17526.master.001.patch
>
>
> Queue#setSuspended() is not used anywhere, probably because when queue 
> wait/wakes on an event, it gets removed or added back to the fairq.
> Removing state, functions, and uses of isSuspended()



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

Reply via email to