Wrap with a for loop while sending&receiving header

[zhichao-li]

Sometimes the stream socket might input or output fewer bytes than requested due to the buffer limits or interruption.Normally we would wrap the read & write socket methods within a loop to remedy the problem of "ERROR: Read partial messageheader" which would be triggered occasionally while running CaffeOnSpark via tcp on 8 nodes.

[yahoocla]

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄

[yahoocla]

CLA is valid!

[mriduljain]

Will need to check as we haven't got a problem unless your network has
issues. Anyway thanks for the pull request, will take a look

On Tue, Jun 28, 2016 at 12:09 AM, li,zhichao notifications@github.com
wrote:

Sometimes the stream socket might input or output fewer bytes than
requested due to the buffer limits or interruption.Normally we would wrap
the read & write socket methods within a loop to remedy the problem of
"ERROR: Read partial messageheader" which would be triggered occasionally

while running CaffeOnSpark via tcp on 8 nodes.

You can view, comment on, or merge this pull request online at:

#98
Commit Summary

• wrap read and write with for loop

File Changes

• M caffe-distri/src/main/cpp/util/socket.cpp
  https://github.com/yahoo/CaffeOnSpark/pull/98/files#diff-0 (40)

Patch Links:

• https://github.com/yahoo/CaffeOnSpark/pull/98.patch
• https://github.com/yahoo/CaffeOnSpark/pull/98.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#98, or mute the thread
https://github.com/notifications/unsubscribe/ACCTVToZktXjkgNJ31ZLZhpUBM9Mz_5Gks5qQMjEgaJpZM4I_zgi
.

[mriduljain]

+1