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

Jing Zhao commented on HADOOP-13227:
------------------------------------

Thanks a lot for updating the patch, Nicholas!

bq. We don't want to read the non-head elements from the queue since it is an 
O( n) operation, where n is the size of the queue.

Can we use ConcurrentLinkedQueue's iterator for the scanning here? We will not 
get any {{ConcurrentModificationException}}, also only the monitor thread 
removes elements from the queue, all other threads only add elements into the 
queue.

> AsyncCallHandler should use a event driven architecture to handle async calls
> -----------------------------------------------------------------------------
>
>                 Key: HADOOP-13227
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13227
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: io, ipc
>            Reporter: Tsz Wo Nicholas Sze
>            Assignee: Tsz Wo Nicholas Sze
>         Attachments: c13227_20160602.patch, c13227_20160606.patch
>
>
> This JIRA is to address [Jing's 
> comments|https://issues.apache.org/jira/browse/HADOOP-13226?focusedCommentId=15308630&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15308630]
>  in HADOOP-13226.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to