[SOLUCIONADO] PostrgreSQL no cierra conexiones en JavaEE

[Hyemin]

Hola, tengo un método que abre una conexion para devolver un resultado con el que se genera un reporte, el problema es que veo en el monitor de PostgreSQL que las conexiones se abren y no se cierran nunca.

Llega a un punto en donde el server directamente se muere aunque cierre el entitymanager creado.

Agradezco cualquier dato o pista de por donde seguir

Código Java:

public Integer obtenerNumero() {
        EntityManager em = emf.createEntityManager();
        Object obj = null;
        
 
        try {
            Query qry = em.createQuery("SELECT MAX(p.numero) FROM Numero p");
            obj = qry.getSingleResult();
            if (obj == null) {
                return 0;
            }
            return (Integer) obj;
        } catch (Exception e) {
            System.out.printf(e.getMessage());
        } finally {
             if (em != null) {
                em.close();
            }
 
       
 
        }
         return (Integer) obj;
    }
 

[Profesor_Falken]

Buenas,

No se puede saber con la infomacion aportada, pero me temo que estas obteniendo el entityManager a partir de una transaction JTA global. Puedes indicar de donde obtienes la factoria (emf) o si usas EJB3, JTA, etc?

Un saludo

[Hyemin]

La factoria la creo en la clase principal

Ahora lo que hice es agregarle emf.close(); en el finally y ya no veo las conexiones colgando.

No se si es la opción mas elegante pero parece servir

Código Java:

public class EstadisticaPersistenciaImpl implements EstadisticaPersistenciaControlador {
 
EntityManagerFactory emf = Persistence.createEntityManagerFactory("PersistenceUnit");
 
 
public Integer obtenerNumero() {
        EntityManager em = emf.createEntityManager();
        Object obj = null;
        
 
        try {
            Query qry = em.createQuery("SELECT MAX(p.numero) FROM Numero p");
            obj = qry.getSingleResult();
            if (obj == null) {
                return 0;
            }
            return (Integer) obj;
        } catch (Exception e) {
            System.out.printf(e.getMessage());
        } finally {
             if (em != null) {
                em.close();
            }
 
       
 
        }
         return (Integer) obj;
    }
 

[Profesor_Falken]

Buenas,

Si como dices en el titulo utilizas Java EE, lo correcto seria que utilices inyeccion de dependencias para no tener que gestionar y cerrar asi tu factory directamente.

Código Java:

@PersistenceUnit(unitName="PersistenceUnit")
private EntityManagerFactory emf;
 

De esta forma sera el propio contenedor el que se encargue de hacer el close y liberar los recursos cuando proceda.

Tambien puedes hacer lo mismo directamente con el entityManager:

Código Java:

@PersistenceContext(unitName="PersistenceUnit")
private EntityManager em;
 

Un saludo

[Hyemin]

Gracias;

Normalmente usaría CDI, el problema es que estoy trabajando sobre el sistema de otra persona y la empresa usa Tomcat7 que no soporta CDI.

Si no tengo chance de usar CDI es muy "sucio" cerrar el factory? o es la unica solucion posible?

[Profesor_Falken]

Buenas,

Yo creo que es correcto, pero hay que tener en cuenta que la creacion de la factoria es bastante costosa, por lo que lo mejor seria abrir un factory, despues hacer todas las consultas que tengas que hacer usando entitymanagers y cerrar al final el factory antes de devolver la informacion al usuario.

Si lo quieres mejorar, la mejor manera de hacerlo en un Tomcat seria usar un listener que controle la creacion y cierre del factory como indican aqui, por ejemplo: http://stackoverflow.com/questions/7...managerfactory

Código Java:

@WebListener
public class EMF implements ServletContextListener {
 
    private static EntityManagerFactory emf;
 
    @Override
    public void contextInitialized(ServletContextEvent event) {
        emf = Persistence.createEntityManagerFactory("PersistenceUnit");
    }
 
    @Override
    public void contextDestroyed(ServletContextEvent event) {
        emf.close();
    }
 
    public static EntityManager createEntityManager() {
        if (emf == null) {
            throw new IllegalStateException("Context is not initialized yet.");
        }
 
        return emf.createEntityManager();
    }
 
}
 

Y despues siempre que necesites hacer una consulta:

Código Java:

EntityManager em = EMF.createEntityManager();
// ...
 

Un saludo

[Hyemin]

Muchas gracias ProfessorFalken

El enlace que envías esta muy bueno :) como siempre, tus aportes son de la mejor calidad