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

System.Data.Common obsolete binary serialization format #39397

Closed
wants to merge 1 commit into from

Conversation

Wraith2
Copy link
Contributor

@Wraith2 Wraith2 commented Jul 16, 2020

closes #39289

/cc @roji

@ghost
Copy link

ghost commented Jul 16, 2020

Tagging subscribers to this area: @roji, @ajcvickers
Notify danmosemsft if you want to be subscribed.

@jkotas
Copy link
Member

jkotas commented Jul 16, 2020

@GrabYourPitchforks
Copy link
Member

Thanks for your enthusiasm. :)

But I don't think the original issue was approved just yet. The original issue was tracking eventual removal, and the suggestion to obsolete these APIs was a "well, here's one way you could consider addressing it." It would need to be part of a larger discussion re: compatibility promises and when we'd want to start ripping this logic out. To be honest I don't expect it to take place this calendar year.

Though the System.Data team is in charge of their own destiny and they can move timelines forward or back as they see fit.

@Wraith2
Copy link
Contributor Author

Wraith2 commented Jul 18, 2020

I thought it was going into net50 so would need to be done pretty quickly. As it's not approved yet i'll close.

@Wraith2 Wraith2 closed this Jul 18, 2020
@karelz karelz added this to the 5.0.0 milestone Aug 18, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Obsolete SerializationFormat.Binary in DataTable/DataSet and put it behind an appcontext switch
5 participants