[ 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