Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Sun, 26 Feb 2017 00:44:44 +0000 (UTC)
From: "Appy (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.13046223.1487990633000.65585.1488069884855@Atlassian.JIRA>
In-Reply-To: <JIRA.13046223.1487990633000@Atlassian.JIRA>
References: <JIRA.13046223.1487990633000@Atlassian.JIRA> <JIRA.13046223.1487990633621@jira-lw-us.apache.org>
Subject: [jira] [Comment Edited] (HBASE-17699) Fix TestLockProcedure
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Sun, 26 Feb 2017 00:44:51 -0000


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

Appy edited comment on HBASE-17699 at 2/26/17 12:43 AM:
--------------------------------------------------------

bq. In the MasterProcedureScheduler#clear, should you call the super#clear? Fix on commit.
There is no super#clear. There's no implementation of that function in AbstractProcedureScheduler.

Updated javadoc for ProcedureScheduler#clear() to mention it's real purpose. That'll clear the confusion around comment.

bq. Should we clear table and server queue and runqueues in the MasterProcedureScheduler when #clear is called?
It already does by calling clearQueue() which cleans up runqueues, tableMap and serverBuckets.

bq. The clearQueue() is useless for AbstractProcedureScheduler. Should it be removed?
Good point. Removed.

Waiting for a day before committing in case anyone wants to followup.


was (Author: appy):
bq. In the MasterProcedureScheduler#clear, should you call the super#clear? Fix on commit.
There is no super#clear. There's no implementation of that function in AbstractProcedureScheduler.

Updated javadoc for ProcedureScheduler#clear() to mention it's real purpose. That'll clear the confusion around comment.

bq. Should we clear table and server queue and runqueues in the MasterProcedureScheduler when #clear is called?
It already does by calling clearQueue() which cleans up runqueues, tableMap and serverBuckets.

bq. The clearQueue() is useless for AbstractProcedureScheduler. Should it be removed?
Good point. Removed.

> Fix TestLockProcedure
> ---------------------
>
>                 Key: HBASE-17699
>                 URL: https://issues.apache.org/jira/browse/HBASE-17699
>             Project: HBase
>          Issue Type: Bug
>          Components: proc-v2
>            Reporter: Appy
>            Assignee: Appy
>            Priority: Blocker
>         Attachments: HBASE-17699.master.001.patch, HBASE-17699.master.002.patch
>
>
> TestLockProcedure is failing consistently after HBASE-17605. It's interesting that HadoopQA didn't report any test failures on that jira. Anyways, need to fix the test now.


--
This message was sent by Atlassian JIRA
(v6.3.15#6346)