Oracle не удаляет курсоры после закрытия результирующего набора

Примечание: мы повторно используем одно соединение.

************************************************
public Connection connection() {        
    try {
        if ((connection == null) || (connection.isClosed()))
        {
            if (connection!=null)
                log.severe("Connection was closed !");
            connection = DriverManager.getConnection(jdbcURL, username, password);
        }
    } catch (SQLException e) {
        log.severe("can't connect: " + e.getMessage());
    }
    return connection;        
}
**************************************************

public IngisObject[] select(String query, String idColumnName, String[] columns) {
    Connection con = connection();

    Vector<IngisObject> objects = new Vector<IngisObject>();
    try {
        Statement stmt = con.createStatement();

        String sql = query;
        ResultSet rs =stmt.executeQuery(sql);//oracle increases cursors count here
        while(rs.next()) {
            IngisObject o = new IngisObject("New Result");
            o.setIdColumnName(idColumnName);            
            o.setDatabase(this);
            for(String column: columns)
                o.attrs().put(column, rs.getObject(column));
            objects.add(o);
        }

        rs.close();// oracle don't decrease cursor count here, while it's expected
        stmt.close();
    } 
    catch (SQLException ex) {
        System.out.println(query);
        ex.printStackTrace();
    }

4 ответов


init.параметр Ора open_cursors определяет максимум открытых курсоров, которые может иметь сеанс сразу. Имеет значение по умолчанию 50. Если приложение превышает это число, возникает ошибка "ORA-01000: превышен максимальный открытый курсор".

поэтому обязательно закрывать ресурсы JDBC, когда они больше не нужны, в частности java.язык SQL.ResultSet и java.язык SQL.Заявление. Если они не закрыты, приложение имеет утечку ресурсов.

в случае повторно используя объект Connection, вы должны знать о том, что открытые курсоры oracle остаются открытыми и используются до тех пор, пока существует соединение и транзакция не закончилась. Когда приложение фиксируется, открытые курсоры освобождаются.

поэтому как дизайнер приложений вам нужно знать приблизительную оценку необходимых открытых курсоров для вашей самой сложной транзакции.

трудность заключается в неспособности внутреннего параметра oracle представления (v$open_cursor, v$sesstat, et. Эл.) чтобы показать разницу между открытыми курсорами, которые являются многоразовыми и открытыми курсорами, которые все еще заблокированы (не многоразовые!) по незамкнутого ResulSet или заявление. Если вы закроете все операторы и объекты ResultSet в блоке finally, ваше приложение будет в полном порядке.

настройки инициализации.параметр ora работает следующим образом (нашему приложению требуется максимум 800 курсоров)

ALTER SYSTEM SET open_cursors = 800 SCOPE=BOTH;

правильный способ сделать это-закрыть каждый ресурс в блоке finally в своем собственном блоке try/catch. Обычно я использую статический служебный класс следующим образом:

public class DatabaseUtils
{
    public static void close(Connection connection)
    {
        try
        {
            if (connection != null)
            {
                connection.close();
            }
        }
        catch (SQLException e)
        {
            // log exception here.
        }
    }

    // similar methods for ResultSet and Statement
}

поэтому я бы написал ваш код так:

public IngisObject[] select(String query, String idColumnName, String[] columns) {

Vector<IngisObject> objects = new Vector<IngisObject>();

Connection con = null;
Statement stmt = null;
ResultSet rs = null;

try 
{
    connection = connection();
    stmt = con.createStatement();

    // This is a SQL injection attack waiting to happen; I'd recommend PreparedStatemen
    String sql = query;
    rs =stmt.executeQuery(sql);//oracle increases cursors count here
    while(rs.next()) 
    {
       IngisObject o = new IngisObject("New Result");
       o.setIdColumnName(idColumnName);            
       o.setDatabase(this);
       for(String column: columns) o.attrs().put(column, rs.getObject(column));
       objects.add(o);
    }

} 
catch (SQLException ex) 
{
    System.out.println(query);
    ex.printStackTrace();
}
finally
{
    DatabaseUtils.close(rs);
    DatabaseUtils.close(stmt);
    DatabaseUtils.close(con);
}

обычно вы помещаете утверждения close для вашего ResultSet и Statement в finally блок, чтобы гарантировать, что они вызываются, даже если происходит исключение (может быть проблема, которую вы здесь имеете). В текущем коде, если SQLException происходит, то два вызова метода close () никогда не будут происходить и курсоры будут оставлены открытыми.

и какой запрос вы используете в Oracle, чтобы увидеть количество открытых курсоров?

Edit:
Этот код должен закрываем курсор. Если это не так, вы должны быть в состоянии увидеть корреляцию 1 к 1 вызова вашего метода и счетчика курсора, идущего вверх на 1. Убедитесь, что нет какого-то неожиданного процесса, который вызывает увеличение количества курсоров.

Если у вас есть права, вы можете запустить этот запрос к базе данных, чтобы увидеть открытый счетчик курсоров sid, чтобы узнать, может быть, это какой-то другой процесс, который увеличивает курсоры, а не ваш конкретно. Он будет отступать с большим, чем 10 открытый курсор, вы можете поднять это, чтобы отфильтровать шум или сузить его специально по имени пользователя или osuser:

select oc.sid,
       count(*) numCur,
       s.username username,
       s.osuser osuser,
       oc.sql_text,
       s.program
  from v$open_cursor oc,
       v$session s
 where s.sid = oc.sid
group by oc.sid, 
         oc.sql_text, 
         s.username, 
         s.osuser, 
         s.program
having count(*) > 10
order by oc.sid;

другой запрос, который может быть полезен, в случае, если несколько sid используют одну и ту же строку запроса, поэтому выше не раскрывает преступника хорошо:

 select oc.sql_text, count(*) 
   from v$open_cursor oc 
   group by oc.sql_text 
   having count(*) > 10 
   order by count(*) desc;

У меня была такая же проблема и я нашел, что-если вы Не закрывайте соединение (потому что вы, возможно, будете использовать его позже) - вам, по крайней мере, нужно сделать подключение.rollback () или подключение.commit () чтобы освободить открытые курсоры вместе с закрытием ResultSet и операторов.