Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

V3 client avoid stream copy for synchroneous requests. #656

Closed
wants to merge 7 commits into from
Closed

V3 client avoid stream copy for synchroneous requests. #656

wants to merge 7 commits into from

Commits on Aug 1, 2016

  1. Removed in-memory response steam copying from the synchroneous Execut… 

    …Query path. The http response stream is now used directly. Disposing is deferred untill enumeration has completed.
    @uffelauesen
    uffelauesen committed Aug 1, 2016
    Copy the full SHA
    9a89de2 View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2016

  1. Trying to allign line endings with original

    @uffelauesen
    uffelauesen committed Aug 2, 2016
    Copy the full SHA
    dcf6802 View commit details
    Browse the repository at this point in the history

  2. Trying to allign line endings with original.

    @uffelauesen
    uffelauesen committed Aug 2, 2016
    Copy the full SHA
    d414136 View commit details
    Browse the repository at this point in the history

Commits on Sep 5, 2016

  1. Review taken into account. Added an overloaded constructor to handle … 

    …this new stream execution path to avoid changes to BatchSaveResult and SaveResult. Also fixed Dispose logic in HttpWebResponseMessage to ensure underlyingResponseMessage isn't disposed twice.
    @uffelauesen
    uffelauesen committed Sep 5, 2016
    Copy the full SHA
    6b24f1e View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2016

  1. Review taken into account. QueryResult are no longer IDisposable inst… 

    …ead the internal ExecuteQuery now returns the constructed IODataResponseMessage for callers to dispose off when needed (only $count queries need this).
    @uffelauesen
    uffelauesen committed Sep 6, 2016
    Copy the full SHA
    d94779b View commit details
    Browse the repository at this point in the history

Commits on Sep 8, 2016

  1. Review taken into account. ExecuteQuery now disposes the message expl… 

    …icit in special cases where the response is NoContent.
    @uffelauesen
    uffelauesen committed Sep 8, 2016
    Copy the full SHA
    7b9d29a View commit details
    Browse the repository at this point in the history

Commits on Sep 14, 2016

  1. Added option boolean AllowDirectNetworkStreamReading to DataServiceCo… 

    …ntext to enable/disable direct stream reading. Default is false.
    @uffelauesen
    uffelauesen committed Sep 14, 2016
    Copy the full SHA
    cf4e196 View commit details
    Browse the repository at this point in the history