Java и течь памяти

Покажите код.

Если честно, то стыдно даже этот быдло код показывать:(
Чтобы хоть табуляция была, то скинул сами файлы, в main там циклы, в dbselect там селекты.
http://file.sibnet.ru/get/file/?id=832844

Вы не закрываете ResultSet-ы и Statement-ы. Перепишите DBSelect так, что бы он возвращал конечные данные, а не ResultSet-ы. Ловите SQLException в DBSelect при помощи try-catch-finally. Закрывайте Statement-ы в finally. При этом ResultSet-ы будут закрываться автоматически.

Спасибо. попробую.

Просто я думал раз метод отработал и выходит то гарбишь корректор собирает эти резалтсеты.

> Просто я думал раз метод отработал и выходит то гарбишь корректор собирает эти резалтсеты.

На них всё ещё есть ссылки из других мест. Поэтому GC не может их удалить.

Просто надо почитывать документацию.

Спасибо большое, буду пробовать.

Какбе в документации нетривиально написано что GC может никогдаих и не собрать. И что есть метод close кой надо вызывать.

Благодарю за цитаты доков! Буду знать!:)

код не смотрел - но краем глаза видел, что работа с bd, посмотри закрываешь ли ты курсоры и статменты после использования, первое что приходит в голову в таких случаях ;)

> Какбе в документации нетривиально написано что GC может никогдаих и не собрать. И что есть метод close кой надо вызывать.

Опять потролить пришёл? Про close() я уже говорил. Дальше я объяснил почему без close() GC их не соберёт. В каком месте у тебя случился дислективный разрыв?

Какбе в документации нетривиально написано что GC может никогдаих и не собрать. И что есть метод close кой надо вызывать.

А вот отсюда по подробнее. Вот кусок из официальной документации

4.1.4 Closing Statements

Statement objects will be closed automatically by the Java garbage collector. Nevertheless, it is recommended as good programming practice that they be closed explicitly when they are no longer needed. This frees DBMS resources immediately and helps avoid potential memory problems. 

Тут вроде ясно сказано, что GC должен собрать незакрытый Statement, если конечно ты сам его не удерживаешь. Не надо путать хороший тон программирования (...as good programming practice...) и обязаловку.

Мне кажется тут проблема не в памяти, а других видах ресурсов. Скажем исчерпан лимит файловых дескрипторов или чего-то такого. Фиг его знает что там творится внутри.

Я htop'ом смотрю как в течение 30 секунд оно у меня сжирает память 400метров:)

>Я htop'ом смотрю как в течение 30 секунд оно у меня сжирает память 400метров:)

И сколько остается свободной для Явы памяти в момент OutOfMemory? Ноль?

Хз, htop такое не показывает или я не знаю где смотреть.

Вы смотрите текст старой спецификации. Вот как это прописано в JDBC 4.0:

13.1.4 Closing Statement Objects

An application calls the method Statement.close to indicate that it has finished
processing a statement. All Statement objects will be closed when the connection
that created them is closed. However, it is good coding practice for applications to
close statements as soon as they have finished processing them. This allows any
external resources that the statement is using to be released immediately.

Открытые файловые дискрипторы можно посмотреть программой lsof. Пердварительно прочтите man lsof.

>Вы смотрите текст старой спецификации. Вот как это прописано в JDBC 4.0:

Прочитал данный текст. Там вроде ничего не написано по части поведения GC по отношению к незакрытому Statement. То, что явное закрытие это «good coding practice», это и ежу понятно.

Вопрос то в другом.

Почему GC не закрыл Statement? И в каких случаях это может произойти? Может в данном случае GC просто не запускался, так как исчерпалась не память, а что-то другое?

Тут либо какой-нибудь Connection неявно от программиста удерживает ссылку на Statement, либо исчерпались другие ресурсы.

Тут вроде ясно сказано, что GC должен собрать незакрытый Statement

Там (в доках) есть еще слова про то что GC может никогда не собрать определенный объект, именно про то и речь.

Уважаемый тролль, перечитайте мой пост еще раз.

>Прочитал данный текст. Там вроде ничего не написано по части поведения GC по отношению к незакрытому Statement. То, что явное закрытие это «good coding practice», это и ежу понятно.

Как же, написано:

An application calls the method Statement.close to indicate that it has finished processing a statement. All Statement objects will be closed when the connection that created them is closed.

Таким образом, незакрытые Statement'ы закрываются после закрытия Connection, но лучше закрывать их вручную. Но Connection в том коде вообще не закрывается, то есть и Statement, по идее, остаются открытыми до конца работы VM.

>Как же, написано:

An application calls the method Statement.close to indicate that it has finished processing a statement. All Statement objects will be closed when the connection that created them is closed.

Ну там вообще-то сказано, что при закрытии Connection закрываются Statement, не более того. Там не сказано, что GC гарантировано не может собрать незакрытые Statement до закрытия Connection. Может Connection использует слабые ссылки на Statement?

Всем огромное спасибо, особенно bbk123. Переписал с ресалтсетов на очередь и чищу ресалсеты и статементы и память больше не текет:) И работает оч быстро.

> Тут либо какой-нибудь Connection неявно от программиста удерживает ссылку на Statement...

Именно об этом я и говорил.

Обратитесь наконец к врачу. Ваши дислективные припадки уже надоели.