You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@avro.apache.org by jcustenborder <gi...@git.apache.org> on 2016/09/12 01:55:11 UTC

[GitHub] avro pull request #123: Avro 1910 - HttpTransceiver buffering

GitHub user jcustenborder opened a pull request:

    https://github.com/apache/avro/pull/123

    Avro 1910 - HttpTransceiver buffering

    Corrected build errors of Avro.perf. Added ReadFully method to ensure that buffers are totally populated. Changed to stream 64k buffers instead of the entire buffer in one call.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jcustenborder/avro AVRO-1910

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/avro/pull/123.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #123
    
----
commit c88f1f448c05a040de1018e5322b0d6958d7b399
Author: Jeremy Custenborder <jc...@gmail.com>
Date:   2016-09-12T01:09:11Z

    Added ReadFully method to ensure that a buffer is fully read. Changed ReadBuffers to allocate a MemoryStream
    with the requested size. Instead of reading the entire large message in a single call we now read 64k blocks
    with ReadFully.

commit 3612f8a4c92dadc680c8a9f147ac9a61100a6431
Author: Jeremy Custenborder <jc...@gmail.com>
Date:   2016-09-12T01:47:07Z

    Corrected formatting.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---