Re: Review Request: Improve concurrency of putMsg / putMsgList

2012-04-24 Thread Claudio Martella
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4852/#review7169 --- Looks good to me, wound't hurt to see some stress test to check

Re: Review Request: Improve concurrency of putMsg / putMsgList

2012-04-24 Thread Bo Wang
On 2012-04-24 20:53:33, Avery Ching wrote: http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/comm/BasicRPCCommunications.java, lines 776-777 https://reviews.apache.org/r/4852/diff/1/?file=104059#file104059line776 Bo, I'm a little leery about

Re: Review Request: Improve concurrency of putMsg / putMsgList

2012-04-24 Thread Avery Ching
On 2012-04-24 20:53:33, Avery Ching wrote: http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/comm/BasicRPCCommunications.java, lines 776-777 https://reviews.apache.org/r/4852/diff/1/?file=104059#file104059line776 Bo, I'm a little leery about

Re: Review Request: Improve concurrency of putMsg / putMsgList

2012-04-24 Thread Avery Ching
On 2012-04-24 20:53:33, Avery Ching wrote: http://svn.apache.org/repos/asf/incubator/giraph/trunk/src/main/java/org/apache/giraph/comm/BasicRPCCommunications.java, lines 776-777 https://reviews.apache.org/r/4852/diff/1/?file=104059#file104059line776 Bo, I'm a little leery about