Skip to content

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

Closed
wants to merge 4 commits into from
Closed

NH-3606 - A method to create a child stateless session #314

wants to merge 4 commits into from

Conversation

rjperes
Copy link
Member

@rjperes rjperes commented Sep 1, 2014

@oskarb oskarb changed the title NH-3606 NH-3606 Allow Creating of Child Stateless Session NH4.1 Sep 2, 2014
@hazzik hazzik added this to the 4.1.0 milestone Sep 21, 2014
- 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)
@rjperes
Copy link
Member Author

rjperes commented Oct 15, 2014

Done.

@hazzik
Copy link
Member

hazzik commented Oct 15, 2014

Yep, much nicer. Thanks

@hazzik hazzik changed the title NH-3606 Allow Creating of Child Stateless Session NH4.1 NH-3606 - Allow Creating of Child Stateless Session NH4.1 Apr 22, 2015
@hazzik hazzik changed the title NH-3606 - Allow Creating of Child Stateless Session NH4.1 NH-3606 - A method to create a child stateless session Apr 22, 2015
@hazzik hazzik modified the milestones: 4.2.0, 4.1.0 Aug 12, 2016
@hazzik hazzik modified the milestones: 4.2.0, 5.0.0 Dec 1, 2016
@fredericDelaporte
Copy link
Member

May I rebase this PR?

@rjperes
Copy link
Member Author

rjperes commented Apr 5, 2017

OK with me.

@@ -13,6 +13,14 @@ namespace NHibernate.Linq
{
public static class LinqExtensionMethods
{
public static IQueryable<T> AsStateless<T>(this IQueryable<T> query)
Copy link
Member

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.

Copy link
Member

@hazzik hazzik left a 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.

@hazzik
Copy link
Member

hazzik commented Sep 1, 2017

@fredericDelaporte is this still required?

@hazzik hazzik modified the milestones: 5.0, 5.1 Sep 1, 2017
@fredericDelaporte
Copy link
Member

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.)

@fredericDelaporte
Copy link
Member

Replaced by #1520

@fredericDelaporte fredericDelaporte removed this from the 5.1 milestone Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants