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

Clarify the difference between System.Data.SqlClient and Microsoft.Data.SqlClient in NuGet #1963

Closed
ericstj opened this issue Mar 23, 2023 · 4 comments
Assignees
Labels
Area\Documentation Issue that requires changes in public documentations/samples.

Comments

@ericstj
Copy link
Member

ericstj commented Mar 23, 2023

@sharwell raised the concern that he couldn't tell the difference between
https://www.nuget.org/packages/System.Data.SqlClient
and
https://www.nuget.org/packages/Microsoft.Data.SqlClient

Which should he use, and why?

Describe the solution you'd like

Clear messaging in the package descriptions around which package to use. Potentially marking the old one as obsolete.

@lcheunglci
Copy link
Contributor

lcheunglci commented Mar 23, 2023

Hi @ericstj,

You can refer to this blog article when Microsoft.Data.SqlClient was introduced.

We would recommend using the latest LTS version of Microsoft.Data.SqlClient, which is currently 5.1.0 as it's the latest stable version that will be actively maintained for another 3 years, which can be found in the Life Cycle page. If you were previously using System.Data.SqlClient, we also have a guide to ease that process. You can also refer to the FAQ. Hope that helps with your decision in which library to use.

@ericstj
Copy link
Member Author

ericstj commented Mar 23, 2023

I'm aware of those sources. I believe @sharwell's request was to make the guidaince more visible for the user who's just coming through NuGet.

For example: https://www.nuget.org/packages?q=SqlClient
image

From this it's not clear what to choose, as both have nearly identical descriptions.

@lcheunglci
Copy link
Contributor

Thanks @ericstj for the clarification. I'll bring it up to the team and see if we can update the nuget description to make it easier to distinguish between them.

@lcheunglci lcheunglci added 🆕 Triage Needed For new issues, not triaged yet. Area\Documentation Issue that requires changes in public documentations/samples. and removed ❔ Question 🆕 Triage Needed For new issues, not triaged yet. labels Mar 23, 2023
@JRahnama
Copy link
Contributor

Closing as it was fixed by #1973

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area\Documentation Issue that requires changes in public documentations/samples.
Projects
None yet
Development

No branches or pull requests

4 participants