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 и операторов.