Visitar URL original
Work towards merging SqlDataReader · Issue #2965 · dotnet/SqlClient · GitHub
Skip to content

Work towards merging SqlDataReader #2965

Closed

Activity

  1. edwardneal commented on Nov 2, 2024

    @edwardneal
    Contributor

    Besides the coding style/comments/CERs, there's other work:

  2. edwardneal commented on Jan 2, 2025

    @edwardneal
    Contributor

    After merging #2967, the diff between netfx and netcore's SqlDataReader implementation is now -407/+159. Most of this is now taken up by two points:

    #3042 is a notable style change though. Once that's merged, I can port the exception handling across as time permits (unless someone else wants to pick this up sooner.) I'll submit a port of PR 499 (and any remaining dependencies/post-PR changes.)

    The rest of the differences are just differences in logging text and debug assertions, along with ~10 lines of behavioural changes in TrySetMetaData and IsDBNull.

  3. moved this from To triage to Done in SqlClient Boardon Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Common Project 🚮Things that relate to the common project project

    Type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions