Cita:
Iniciado por Lino-kun 
Hola.
Esta clase se hace la peticion al pool que ya esta configurado en el servidor.
...
No te lo tomes a mal pero el código que has puesto no es eficiente ni seguro. No es eficiente por que estas haciendo un "JNDI lookup" cada vez que pides una conexion, lo cual no es eficiente si siempre vas a usar el mismo DataSource, y ademas estas repitiendo mucho código en el metodo comun que total vas a hacer algo parecido en cada metodo. Y no es seguro por que asi como lo usas no garantizas que las conexiones siempre se devuelvan al pool.
Aparte de que estas perdiendo mucha informacion de las excepciones al encapsularlas solo copiando el getMessage, y enmascarando el origen por otro que quiza no es (convirtiendo un NameNotFound en una SQLException, por ejemplo)
Lo digo solo como critica constructiva por que no sería muy recomendable usar ese codigo en produccion, IMHO.
S!