NanoContainer
  1. NanoContainer
  2. NANO-101

In FailoverSessionDelegator when you try to get a Session lazilly you should throw an HibernateException insted of PicoInitializationException.

    Details

    • Type: Wish Wish
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 1.0-beta-3
    • Component/s: hibernate
    • Labels:
      None
    • Number of attachments :
      0

      Description

      In FailoverSessionDelegator when you try to get a Session lazily you should throw an HibernateException insted of PicoInitializationException.

      Its because my component does not know that Pico exists and if I cant open a session, even I do it lazily, because I had a problem with Hibernate, an Hibernate excpetion should be throwed to be treated as an HibernateException.

      The line numeber is 50:
      http://cvs.picocontainer.codehaus.org/viewrep/picocontainer/java/hibernate/src/java/org/nanocontainer/hibernate/FailoverSessionDelegator.java?r=1.2#l50

      Thanks,
      Jose Luiz Peleteiro

        Activity

        Hide
        Jose Peleteiro added a comment -

        Forget it... I dont know why, but in my mind HibernateException was an RuntimeException.

        Show
        Jose Peleteiro added a comment - Forget it... I dont know why, but in my mind HibernateException was an RuntimeException.
        Hide
        Jörg Schaible added a comment -

        HibernateException is a checked exception. Change not possible because of underlaying interface of net.sf.hibernate.Session.

        Show
        Jörg Schaible added a comment - HibernateException is a checked exception. Change not possible because of underlaying interface of net.sf.hibernate.Session.
        Jörg Schaible made changes -
        Field Original Value New Value
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Won't Fix [ 2 ]
        Fix Version/s 1.0-beta-3 [ 11242 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Jose Peleteiro
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: