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

New NuGet package names based on the new naming conventions #3544

Closed
wants to merge 4 commits into from

Conversation

attilah
Copy link
Contributor

@attilah attilah commented Oct 13, 2017

New package names for the packages which will not be separated into multiple pieces
Naming follows the new naming conventions
Package names no longer contains duplicated 'Orleans'

@attilah
Copy link
Contributor Author

attilah commented Oct 13, 2017

@dotnet-bot test bvt please

@veikkoeeva
Copy link
Contributor

Aah, hey, now that we're on this patch, Microsoft.Orleans.OrleansSqlUtils would need to be Microsoft.Orleans.SqlUtils, but even better would be Microsoft.Orleans.AdoNetUtils and prepare to streamline mentions of SqlServer etc. to AdoNet where applicable in the code too (in a separate PR).

@attilah
Copy link
Contributor Author

attilah commented Oct 16, 2017

@veikkoeeva the packages which were not renamed as part of this PR is not renamed, since we'll separate it into pieces probably after Beta. They will be split into Membership and Storage in most cases so:

  • Microsoft.Orleans.Membership.Sql
  • Microsoft.Orleans.Storage.Sql

etc...

@sergeybykov
Copy link
Contributor

I think @veikkoeeva is suggesting to name them:

  • Microsoft.Orleans.Membership.Sql -> Microsoft.Orleans.Membership.AdoNet
  • Microsoft.Orleans.Storage.Sql -> Microsoft.Orleans.Storage.AdoNet

Makes sense to me.

@sergeybykov sergeybykov added this to the Triage milestone Oct 16, 2017
@attilah
Copy link
Contributor Author

attilah commented Oct 17, 2017

That naming would suggest to me that we're working with any ADO.NET provider, which is not the case.

@veikkoeeva
Copy link
Contributor

veikkoeeva commented Oct 17, 2017

That naming would suggest to me that we're working with any ADO.NET provider, which is not the case.

Is that opposed to sql? Arguably AdoNet is more to the point than sql as we're working on ADO.NET platform and support potentially any vendor that has ADO.NET implementation. It's also notable Orleans could use (in some deployments should) use other than SQL to work with the storage underneath ADO.NET. This would line the code better in the future so we could rename instances of SqlServer and Sql when we're really working with ADO.NET.

@sergeybykov
Copy link
Contributor

Naming is hard. :-)
Looking again at the alternatives: "ADO.NET" isn't really used as a name in .NET anymore, it's really System.Data; "Relational" seems too broad; "SQL", while not ideal, now seems to me less controversial than "ADO.NET" or "Relational".

@veikkoeeva
Copy link
Contributor

veikkoeeva commented Oct 17, 2017

@sergeybykov I think it's always been System.Data (the namespace and associated libraries), but the architecture name is ADO.NET, as mentioned here, for instance. Or the the latest documentation here. I think this is what defines the underlying platform. I'm OK with even the current mixed bag, but I still think the most in line would be to call it ADO.NET. :)

@sergeybykov
Copy link
Contributor

Okay, we'll go with *.AdoNet then.

Attila Hajdrik added 2 commits November 3, 2017 13:06
# Conflicts:
#	src/Orleans.CodeGeneration/Orleans.CodeGeneration.csproj
@sergeybykov sergeybykov mentioned this pull request Nov 3, 2017
@jdom
Copy link
Member

jdom commented Nov 7, 2017

Before merging, we need to reach some explicit decision on this, as the side-effects of blindly renaming are pretty bad. Here is some examples of what I mean: dotnet/aspnetcore#1222
And of course when searching for Orleans in nuget.org, you'll find tons of packages which are deprectated, intermixed with the latest ones, which creates a lot of noise. This is bad too, although at least a human decides what to do there.
I'm still on the fence of what we should do, but I lean towards not renaming... splitting packages is a different story, but blindly renaming to a slightly better ID I believe is just not worth it due to the side effects

@jdom
Copy link
Member

jdom commented Nov 7, 2017

And here is an open issue regarding the package rename feature: NuGet/Home#1590

@attilah
Copy link
Contributor Author

attilah commented Nov 9, 2017

@jdom PRI2 workitem for them and all connecting discussion is started years ago, I don't think that we should hold off this and waiting for a solution from NuGet. Once they've a solution, we can add that "compat" layer to our publishing strategy.

@sergeybykov
Copy link
Contributor

Closing this because we plan not to change names of the existing packages.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants