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

Matteo Bertozzi commented on HBASE-16642:
-----------------------------------------

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)

Reply via email to