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

migrate to NHibirnate Core #168

Closed
ku3mich opened this issue Dec 8, 2018 · 6 comments
Closed

migrate to NHibirnate Core #168

ku3mich opened this issue Dec 8, 2018 · 6 comments

Comments

@ku3mich
Copy link

ku3mich commented Dec 8, 2018

just proposal - I can this, need only an agree

@ku3mich
Copy link
Author

ku3mich commented Dec 8, 2018

#167 related

@loic-sharma
Copy link
Owner

What’re the benefits of NHibernate? I haven’t used it, so I’m not very familiar with it. EF Core works well for this project, so I don’t feel there’s a need to move something else.

@ku3mich
Copy link
Author

ku3mich commented Dec 9, 2018

The idea to migrate NH is based on the fact I discover that EF Core does not support group by(even EF 2.2 #167 and in further plans, you can investigate this starting from this thread for example https://stackoverflow.com/questions/48082277/entityframework-core-group-by-gives-me-an-order-by)
Also while the investigation I've found that EF Core has also troubles with joins, count, order by. I was very surprised by these facts because EF 6 does it perfectly. From other side I see a lot of new features such as Symbol Server or and taking into consideration fact that primary NuGet server is used to search/retrieve the results of NuGet by various criteria I do not think that EF Core is right ORM(at least at future several years). NH much more flexible and powerful(quality of sql primary) than EF and to avoid further troubles and while we have only two tables in DB I think we need to migrate to NH.

@loic-sharma
Copy link
Owner

I would like to avoid changing our database provider until we absolutely need to. If anything, I would rather move to EF6 than NHibernate, and EF6 will be supported on .NET Core 3, which is currently in preview. Does that seem reasonable? I'll close off this issue for now, but feel free to add any comments.

@ku3mich
Copy link
Author

ku3mich commented Dec 10, 2018

I would like to do not agree that moving back to EF6 is reasonable ( EF6 in compare NHibernate is much more weaker ORM.
From other side NH supports such features as shards, search, caching which may help in cases when 'Pacakges' table contains a thousands of packages.
So migrating now to NH will resolve a lot of future issues.

also for Azure lovers(maybe interesting fact):
https://ayende.com/blog/4178/sql-azure-sharding-and-nhibernate-a-call-for-volunteers

@loic-sharma
Copy link
Owner

loic-sharma commented Dec 10, 2018

NuGet.org uses entity framework and scales to 2 million records without any sharding. I’ve also indexed all of NuGet.org using BaGet on a single database, so I don’t think there’s a need to replace the current solution at this time. I’ll read up more about the caching and searching though, that could be useful!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants