Re: Review Request 32519: Patch for KAFKA-2050

2015-03-29 Thread Neha Narkhede

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32519/#review78167
---

Ship it!


Ship It!

- Neha Narkhede


On March 26, 2015, 2:36 a.m., Tim Brooks wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/32519/
 ---
 
 (Updated March 26, 2015, 2:36 a.m.)
 
 
 Review request for kafka.
 
 
 Bugs: KAFKA-2050
 https://issues.apache.org/jira/browse/KAFKA-2050
 
 
 Repository: kafka
 
 
 Description
 ---
 
 Avoid calling size() on concurrent queue.
 
 
 Diffs
 -
 
   core/src/main/scala/kafka/network/SocketServer.scala 
 76ce41aed6e04ac5ba88395c4d5008aca17f9a73 
 
 Diff: https://reviews.apache.org/r/32519/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Tim Brooks
 




Review Request 32519: Patch for KAFKA-2050

2015-03-25 Thread Tim Brooks

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32519/
---

Review request for kafka.


Bugs: KAFKA-2050
https://issues.apache.org/jira/browse/KAFKA-2050


Repository: kafka


Description
---

Avoid calling size() on concurrent queue.


Diffs
-

  core/src/main/scala/kafka/network/SocketServer.scala 
76ce41aed6e04ac5ba88395c4d5008aca17f9a73 

Diff: https://reviews.apache.org/r/32519/diff/


Testing
---


Thanks,

Tim Brooks



Re: Review Request 32519: Patch for KAFKA-2050

2015-03-25 Thread Sriharsha Chintalapani

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32519/#review77855
---

Ship it!


Ship It!

- Sriharsha Chintalapani


On March 26, 2015, 2:36 a.m., Tim Brooks wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/32519/
 ---
 
 (Updated March 26, 2015, 2:36 a.m.)
 
 
 Review request for kafka.
 
 
 Bugs: KAFKA-2050
 https://issues.apache.org/jira/browse/KAFKA-2050
 
 
 Repository: kafka
 
 
 Description
 ---
 
 Avoid calling size() on concurrent queue.
 
 
 Diffs
 -
 
   core/src/main/scala/kafka/network/SocketServer.scala 
 76ce41aed6e04ac5ba88395c4d5008aca17f9a73 
 
 Diff: https://reviews.apache.org/r/32519/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Tim Brooks