[JIRA-Opensymphony] Commented: (USER-46) HIbernateProviders are buggy.

The following comment has been added to this issue:

     Author: Konstantin Pribluda
    Created: Wed, 6 Aug 2003 6:04 AM
       Body:
I think problem consists of 2 parts: 
- configuration management ( already solved satisfactory )
which provides session factory for those who need it ( 
DAO Objects )
- connection management ( quite catastrofic at the moment ) 

I see no problem in DAO approach - only if they open / close 
sessions. 

I think a good solution would be:
DAO doing the work, and not worrying about  opening / closing sesion
DAO pull session out of some session provider, and gives it back after use. 

Default one could just open / flushclose those session, 
and for those who is interested in more performant solution
there is still a way to implement whatever they like - 
one which provides session out of thread local, one which 
cooperates with request filter and opens / flushcloses on demand. 

Another cool improvement could be using just the same session factory
for properyset ( instead of 3 distinct  ones ), and cascading entity deletes to properties...
---------------------------------------------------------------------
View the issue:

  http://jira.opensymphony.com/secure/ViewIssue.jspa?key=USER-46


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: USER-46
    Summary: HIbernateProviders are buggy.
       Type: Bug

     Status: Assigned
   Priority: Blocker

    Project: OSUser
 Components: 
             ProfileProviders

   Assignee: Patrick Lightbody
   Reporter: Konstantin Pribluda

    Created: Mon, 4 Aug 2003 12:03 PM
    Updated: Mon, 4 Aug 2003 12:03 PM
Environment: Any

Description:
Hmm, OSUserHibernateConfigurationProviderImpl extends 
DefaultHibernateConfigurationProvider - but forgets to call

super.setupConfiguration(). 

This successfully prevents hibernate property set from proper 
( any :) ) function - it just does not get any SessionFactory...



Oh, just forgot , while we are at it - why do we need 3 instances of 
this provider? ( one fo AccessProvider, one for CredentialsProvider... )


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.opensymphony.com/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01