连接未在多线程程序中释放到池中

时间:2017-11-20 16:19:27

标签: java multithreading postgresql jax-ws connection-pooling

大家好

由于某些原因,我的连接没有被释放。在这一天的大部分时间里,我一直坐在这个问题上,所以现在我希望你们中的一个可以帮助我。

DataSource位于Swagger jaxws服务器中。所以在每个请求中我都从池中检索连接。这是我的DataSource类,它从池中返回一个连接:

import java.beans.PropertyVetoException;
import java.io.IOException;
import java.sql.Connection;
import java.sql.SQLException;
import org.apache.commons.dbcp2.BasicDataSource;
/**
 *
 * @author Lagoni
 */
public class DataSource {
    private static DataSource datasource;
    private BasicDataSource ds;
    private DataSource() throws IOException, SQLException, PropertyVetoException {
        ds = new BasicDataSource();
        ds.setDriverClassName("org.postgresql.Driver");
        ds.setUsername("username");
        ds.setPassword("pw");
        ds.setUrl("jdbc:postgresql://host" + 5432 + "/db");

        ds.setMaxWaitMillis(20000); //wait 10 seconds to get new connection
        ds.setMaxTotal(5);
        ds.setMaxIdle(5);
        ds.setTestWhileIdle(true);
        ds.setTestOnReturn(true);
        ds.setTimeBetweenEvictionRunsMillis(1000);
        ds.setSoftMinEvictableIdleTimeMillis(100); 
        ds.setMinEvictableIdleTimeMillis(10); 
        ds.setMaxConnLifetimeMillis(1000*60*10);
    }

    public static DataSource getInstance() throws IOException, SQLException, PropertyVetoException {
        if (datasource == null) {
            datasource = new DataSource();
        }
        return datasource;
    }

    public Connection getConnection() throws SQLException {
        return ds.getConnection();
    }

}

对于需要使用数据库连接的每个函数,通过调用:

来检索它
Connection con = DataSource.getInstance().getConnection();

这是我得到的“无法获得连接,池错误超时等待空闲对象”。我确保每个线程只使用一个连接。因此,如果函数需要对数据库进行多次调用,则它会重用con变量。

我得出结论,我永远不应该调用con.close(),因为它会返回到池的空连接。当连接完成一个函数时,将调用以下函数:

resultSet.close();
statement.close();

在连接被宣布闲置之前,有什么我忘了做的吗?或者我只是没有正确实现连接池?我应该尝试使用其他种类的游泳池吗?

我正在使用以下maven依赖项:

<dependency>
  <groupId>org.apache.commons</groupId>
  <artifactId>commons-dbcp2</artifactId>
  <version>2.1.1</version>
</dependency>
<dependency>
  <groupId>org.apache.commons</groupId>
  <artifactId>commons-pool2</artifactId>
  <version>2.4.3</version>
</dependency>
<dependency>
  <groupId>org.postgresql</groupId>
  <artifactId>postgresql</artifactId>
  <version>42.1.4</version>
</dependency>

编辑1

这会是一个解决方案吗?

   try(Connection con = DataSource.getInstance().getConnection()){
        UploadedFile file = new FileServerImplementation().uploadFile(fileType, fileName, folderId, projectId, tokenString, fileInputStream, con);
        if(file != null){
            return Response.ok().entity(file).build();
        }
    }

这里的uploadFile方法如下:

public UploadedFile uploadFile(String fileType, String fileName, Long folderId, Long projectId, String tokenString, InputStream fileInputStream, Connection con) throws SQLException, IOException, PropertyVetoException{
    IOController ioController = new IOController();
    DatabaseController controller = DatabaseController.getInstance();
    UploadedFile file = null;
    if(ioController.uploadFile(fileType, fileName, controller.getFolderPath(folderId, con), projectId, fileInputStream)){
        file = controller.uploadFile(fileType, fileName, folderId, projectId, con);
    }else{
        System.out.println("Error uploading " + fileName + " to folder!");
    }
    return file;
}

IOController将文件保存到磁盘上,然后方法uploadFile将一些数据上传到数据库。或者我应该每次调用DatabaseController类中的方法从池中获取新连接吗?

解决方案

我最终使用编辑1作为一种方法。我必须确保在没有再关闭它们的情况下我没有创建不必要的连接。

1 个答案:

答案 0 :(得分:2)

使用连接池时,您需要close连接将其释放到池中以便重复使用。 保持线程中的连接。