我们正在使用
开发网站我们遇到了一些连接泄漏并且认为我们已经纠正了它们(数据库不再停止响应),但是连接池的行为似乎仍然是漏洞,因为我们有许多空闲连接大于上下文中设置的maxIdle .XML。我想确定问题是否已解决。
出于测试目的,我使用以下context.xml:
<Resource
auth="Container"
name="jdbc/postgres"
factory="org.apache.tomcat.jdbc.pool.DataSourceFactory"
type="javax.sql.DataSource"
username="admin"
password="..."
driverClassName="org.postgresql.Driver"
url="jdbc:postgresql://127.0.0.1:5432/..."
initialSize="1"
maxActive="50"
minIdle="0"
maxIdle="3"
maxWait="-1"
minEvictableIdleTimeMillis="1000"
timeBetweenEvictionRunsMillis="1000"
/>
如果我理解正确,我们应该在启动时有1个空闲连接,从0到3,具体取决于负载,对吗?
正在发生的事情是:启动时连接1个,负载低时最多3个空闲连接,高负载后3个空闲连接。然后这些连接不立即关闭,我们不知道它们何时/是否将被关闭(有时它们中的一些已关闭)。
所以问题是:这种行为是否正常?
感谢您的帮助
编辑:添加了工厂属性,没有改变问题
编辑2 :使用removeAbandoned&amp; removeAbandonedTimeout使每个removeAbandonedTimeout都关闭空闲连接。所以我们可能仍然有一些连接泄漏。以下是我们用于连接数据库和执行请求的一些代码:
PostgreSQLConnectionProvider ,只是一个提供连接的静态类:
public class PostgreSQLConnectionProvider {
public static Connection getConnection() throws NamingException, SQLException {
String dsString = "java:/comp/env/jdbc/postgres";
Context context = new InitialContext();
DataSource ds = (DataSource) context.lookup(dsString);
Connection connection = ds.getConnection();
return connection;
}
}
DAO 抽象类:
public abstract class DAO implements java.lang.AutoCloseable {
// Private attributes :
private Connection _connection;
// Constructors :
public DAO() {
try { _connection = PostgreSQLConnectionProvider.getConnection(); }
catch (NamingException | SQLException ex) {
Logger.getLogger(DAO.class.getName()).log(Level.SEVERE, null, ex);
}
}
// Getters :
public Connection getConnection() { return _connection; }
// Closeable :
@Override
public void close() throws SQLException {
if(!_connection.getAutoCommit()) {
_connection.rollback();
_connection.setAutoCommit(true);
}
_connection.close();
}
}
UserDAO ,一个小型DAO子类(我们有几个DAO子类来请求数据库):
public class UserDAO extends DAO {
public User getUserWithId(int id) throws SQLException {
PreparedStatement ps = null;
ResultSet rs = null;
User user = null;
try {
String sql = "select * from \"USER\" where id_user = ?;";
ps = getConnection().prepareStatement(sql);
ps.setInt(1, id);
rs = ps.executeQuery();
rs.next();
String login = rs.getString("login");
String password = rs.getString("password");
String firstName = rs.getString("first_name");
String lastName = rs.getString("last_name");
String email = rs.getString("email");
user = new User(id, login, password, firstName, lastName, email);
}
finally {
if(rs != null) rs.close();
if(ps != null) ps.close();
}
return user;
}
}
使用DAO子类的示例:
try(UserDAO dao = new UserDAO()) {
try {
User user = dao.getUserWithId(52);
}
catch (SQLException ex) {
// Handle exeption during getUserWithId
}
}
catch (SQLException ex) {
// Handle exeption during dao.close()
}
答案 0 :(得分:1)
查看代码,看起来连接是在DAO的生命周期中获取的,而不是语句的生命周期,这是通常的期望。通常情况下,您可以在执行语句时从池中获取连接,并在完成后调用close()以将其返回池中。
此外,在您的finally子句中,rs.close()
和ps.close()
都会抛出异常,导致错过针对预准备语句的最后一次调用。
在Java 7中,您还可以使用try with resources语句来关闭准备好的语句和连接。根据规范,当语句关闭时,驱动程序应该为您关闭结果。