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.

Sign up for GitHub

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

Keep tracking the file offset in memory, don't perform expensive Seek calls on every Read|WriteAsync #49145

Closed
wants to merge 20 commits into from
Closed

Keep tracking the file offset in memory, don't perform expensive Seek calls on every Read|WriteAsync #49145

wants to merge 20 commits into from

Commits on Feb 25, 2021

  1. introduce WindowsFileStreamStrategy

    @adamsitnik
    adamsitnik committed Feb 25, 2021
    Configuration menu
    Copy the full SHA
    cd1698d View commit details
    Browse the repository at this point in the history

  2. introduce SyncWindowsFileStreamStrategy

    @adamsitnik
    adamsitnik committed Feb 25, 2021
    Configuration menu
    Copy the full SHA
    ad5b455 View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2021

  1. introduce AsyncWindowsFileStreamStrategy

    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    a5ff09c View commit details
    Browse the repository at this point in the history

  2. some minor improvements after reading the code again

    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    5a64215 View commit details
    Browse the repository at this point in the history

  3. only DerivedFileStreamStrategy needs to have a reference to FileStream 

    remove finalizer from FileStream, keep it only in DerivedFileStreamStrategy and LegacyFileStreamStrategy
    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    e27ce6c View commit details
    Browse the repository at this point in the history

  4. implement ReadByte and WriteByte to make new windows strategies fully… 

    … functional

they can now be used directly without any buffering on top of them!
    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    3f84bd5 View commit details
    Browse the repository at this point in the history

  5. implement FlushAsync for no buffering strategies as nop to get Flush_… 

    …NothingToFlush_CompletesSynchronously passing
    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    402cf65 View commit details
    Browse the repository at this point in the history

  6. use the new strategies when buffering is not enabled

    @adamsitnik
    adamsitnik committed Feb 26, 2021
    Configuration menu
    Copy the full SHA
    9a30387 View commit details
    Browse the repository at this point in the history

Commits on Mar 1, 2021

  1. introduce BufferedFileStreamStrategy

    @adamsitnik
    adamsitnik committed Mar 1, 2021
    Configuration menu
    Copy the full SHA
    f34cb02 View commit details
    Browse the repository at this point in the history

  2. fix the Mono build

    @adamsitnik
    adamsitnik committed Mar 1, 2021
    Configuration menu
    Copy the full SHA
    fb5b611 View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2021

  1. use the Legacy strategy by default for now, add tests for the other i… 

    …mplementation
    @adamsitnik
    adamsitnik committed Mar 2, 2021
    Configuration menu
    Copy the full SHA
    b1decbf View commit details
    Browse the repository at this point in the history

  2. restore old file name to make it easier to review the code (as diff w… 

    …ithin the file, not a removal and addition)
    @adamsitnik
    adamsitnik committed Mar 2, 2021
    Configuration menu
    Copy the full SHA
    8d58437 View commit details
    Browse the repository at this point in the history

  3. Don't set the buffer to null, to avoid a NullReferenceException 

    when users have a race condition in their code (i.e. they call
Close when calling another method on Stream like Read).
    @adamsitnik
    adamsitnik committed Mar 2, 2021
    Configuration menu
    Copy the full SHA
    33e6b8e View commit details
    Browse the repository at this point in the history

  4. fix the browser build?

    @adamsitnik
    adamsitnik committed Mar 2, 2021
    Configuration menu
    Copy the full SHA
    77b6d97 View commit details
    Browse the repository at this point in the history

  5. reverting the flushing changes as it looks that they have introduced … 

    …a bug
    @adamsitnik
    adamsitnik committed Mar 2, 2021
    Configuration menu
    Copy the full SHA
    33c7c83 View commit details
    Browse the repository at this point in the history

Commits on Mar 3, 2021

  1. 0 bytes reads are OK only for FileStream when the read buffer is empty

    @adamsitnik
    adamsitnik committed Mar 3, 2021
    Configuration menu
    Copy the full SHA
    60cd6e7 View commit details
    Browse the repository at this point in the history

  2. simplify the source code by removing some of the optimizations that a… 

    …re not providing too much gains
    @adamsitnik
    adamsitnik committed Mar 3, 2021
    Configuration menu
    Copy the full SHA
    e8d6f77 View commit details
    Browse the repository at this point in the history

Commits on Mar 4, 2021

  1. don't verify OS handle position

    @adamsitnik
    adamsitnik committed Mar 4, 2021
    Configuration menu
    Copy the full SHA
    d06f175 View commit details
    Browse the repository at this point in the history

  2. track the file offset in memory, don't use expensive sys calls to syn… 

    …chronize it
    @adamsitnik
    adamsitnik committed Mar 4, 2021
    Configuration menu
    Copy the full SHA
    91423fa View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2021

  1. there is no need to set the Length since we are now tracking the offs… 

    …et in memory
    @adamsitnik
    adamsitnik committed Mar 5, 2021
    Configuration menu
    Copy the full SHA
    a7ca4cb View commit details
    Browse the repository at this point in the history