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

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

what is the rationale behind the change?
to me looks like there are more objects allocation and the logic around remove 
and wake up of the queue is less clean.

if you want to move forward with the change at least wrap all this stuff in a 
ProcedureDelayedQueue class or something like that, so we don't have to deal 
with creating those DelayedContainer and know about the POISON stuff.

> Use DelayQueue instead of TimeoutBlockingQueue
> ----------------------------------------------
>
>                 Key: HBASE-16642
>                 URL: https://issues.apache.org/jira/browse/HBASE-16642
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Hiroshi Ikeda
>            Priority: Minor
>         Attachments: 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