[
https://issues.apache.org/jira/browse/HBASE-16642?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15574040#comment-15574040
]
Matteo Bertozzi edited comment on HBASE-16642 at 10/14/16 3:29 AM:
-------------------------------------------------------------------
the patch is ok, it's just the for loop with the POISON that is not needed.
I'll take care of it on commit.
{code}
+ for (int i = 0; i < threads.length; ++i) {
+ waitingTimeout.add(DelayedContainer.POISON);
+ }
{code}
was (Author: mbertozzi):
the patch is ok, it's just the for loop with the POISON that is not needed.
I'll take care of it on commit.
+ for (int i = 0; i < threads.length; ++i) {
+ waitingTimeout.add(DelayedContainer.POISON);
+ }
> Use DelayQueue instead of TimeoutBlockingQueue
> ----------------------------------------------
>
> Key: HBASE-16642
> URL: https://issues.apache.org/jira/browse/HBASE-16642
> Project: HBase
> Issue Type: Sub-task
> Components: proc-v2
> Reporter: Hiroshi Ikeda
> Assignee: Matteo Bertozzi
> Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-16642-v2.patch, HBASE-16642-v3.patch,
> HBASE-16642-v4.patch, HBASE-16642.master.V1.patch
>
>
> Enqueue poisons in order to wake up and end the internal threads.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
