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

Feature | Project Microsoft.SqlServer.Server separation phase 2 (MDS consumes MSS using netcore) #1585

Merged
merged 10 commits into from
Jun 14, 2022

Conversation

DavoudEshtehari
Copy link
Contributor

@DavoudEshtehari DavoudEshtehari commented Apr 14, 2022

Dropping the Microsoft.Data.SqlClient.Server namespace for supported types by Microsoft.SqlServer.Server package.

Any attempt to use the common types from Microsoft.SqlServer.Server and System.Data.SqlClient packages are known conflicts.

  • Related issue on dotnet/runtime: #66531

@DavoudEshtehari DavoudEshtehari added the 🆕 Public API Issues/PRs that introduce new APIs to the driver. label Apr 14, 2022
@DavoudEshtehari DavoudEshtehari added this to the 5.0.0-preview3 milestone Apr 14, 2022
@DavoudEshtehari
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@DavoudEshtehari DavoudEshtehari force-pushed the MSqlServerSep2 branch 2 times, most recently from a8c7f91 to 01fa0a5 Compare April 21, 2022 19:25
@DavoudEshtehari DavoudEshtehari marked this pull request as ready for review April 21, 2022 19:25
@cheenamalhotra
Copy link
Member

Hi @DavoudEshtehari

You may want to add dependency in Microsoft.Data.SqlClient.nuspec for non-netfx targets to be able to reference Microsoft.SqlServer.Server released version from external apps.

@Kaur-Parminder

This comment was marked as outdated.

# Conflicts:
#	src/Microsoft.Data.SqlClient/netcore/src/Microsoft.Data.SqlClient.csproj
#	src/Microsoft.Data.SqlClient/src/Microsoft/Data/Common/AdapterUtil.cs
@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #1585 (02f02ff) into main (966fc6e) will increase coverage by 0.05%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main    #1585      +/-   ##
==========================================
+ Coverage   71.28%   71.34%   +0.05%     
==========================================
  Files         296      290       -6     
  Lines       61161    61048     -113     
==========================================
- Hits        43600    43554      -46     
+ Misses      17561    17494      -67     
Flag Coverage Δ
addons 92.38% <ø> (ø)
netcore 74.92% <33.33%> (+0.10%) ⬆️
netfx 69.13% <75.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...core/src/Microsoft/Data/SqlClient/SqlConnection.cs 77.19% <0.00%> (ø)
...SqlClient/src/Microsoft/Data/Common/AdapterUtil.cs 68.38% <ø> (ø)
...ient/src/Microsoft/Data/SqlClient/Server/SqlSer.cs 45.45% <ø> (ø)
...SqlClient/src/Microsoft/Data/SqlClient/SqlEnums.cs 77.37% <ø> (ø)
...lClient/src/Microsoft/Data/SqlClient/SqlUdtInfo.cs 100.00% <ø> (ø)
...etfx/src/Microsoft/Data/SqlClient/SqlConnection.cs 58.80% <75.00%> (ø)
.../netcore/src/Microsoft/Data/SqlClient/TdsParser.cs 73.70% <100.00%> (ø)
...src/Microsoft/Data/SqlClient/SqlMetadataFactory.cs 81.66% <0.00%> (-10.84%) ⬇️
...ActiveDirectoryAuthenticationTimeoutRetryHelper.cs 56.81% <0.00%> (-6.82%) ⬇️
...c/Microsoft/Data/SqlClient/SqlConnectionFactory.cs 68.85% <0.00%> (-0.82%) ⬇️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 966fc6e...02f02ff. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🆕 Public API Issues/PRs that introduce new APIs to the driver.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants