-
Notifications
You must be signed in to change notification settings - Fork 934
NH-3606 - A method to create a child stateless session #314
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
Conversation
- More unit tests - Logic moved from extension method to SessionImpl - Proper disposing of child stateless session by parent session - Added extension method AsStateless (NH-3231)
Done. |
Yep, much nicer. Thanks |
May I rebase this PR? |
OK with me. |
@@ -13,6 +13,14 @@ namespace NHibernate.Linq | |||
{ | |||
public static class LinqExtensionMethods | |||
{ | |||
public static IQueryable<T> AsStateless<T>(this IQueryable<T> query) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like this extensions. It needs to be implemented as a separate feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this needs to be reworked into 5.0 way. Eg. use interface methods instead of extension.
@fredericDelaporte is this still required? |
This NH-3606 feature is not covered by the new "session sharing some parts of another" way of opening sessions, having replaced child sessions. Hibernate Java does not have it either, so having realigned on Hibernate for session opening has not brought this feature in. I guess it could still be useful and added to current master, but this would require another PR. (Or rewriting most changes of this one.) |
Replaced by #1520 |
#910, https://nhibernate.jira.com/browse/NH-3606