Зачем закрывать курсор и соединение к БД?

Question

[wideShift]

Разве по окончании скрипта память не освободиться? Зачем закрывать курсор и соединение к БД?

И еще вопрос: почему если закрыть курсор после соединения, то ошибки не возникает(курсор же использует соединение)? Не логично же, почему при закрытии соединения автоматически курсор не закрывается?

conn.commit()
conn.close()
cursor.close()

Answer 1

[Михаил Трайнин]

Это называется устойчивой хорошей практикой, которая позволяет в будущем избежать большого кол-ва ошибок.

Для начала надо определится с терминологией:

cursor - Класс драйвера, через который происходит взаимодействие с базой данных.
connection - Класс подключения к базе данных.

У любой базы данных есть максимально допустимое кол-во подключений. Как и максимально допустимое кол-во открытых курсоров.

Если не закрывать - вы рано или поздно упретесь в лимит.

Так же, следует помнить, что транзакция начинается до выполнения запроса, и при определенных настройках бд, на момент выполнения транзакции таблицы лочатся, и не дают другим потокам или программам ими воспользоваться. По умолчанию транзакция создаётся до выполнения первого запроса к БД, и все последующие запросы выполняются в контексте этой транзакции. А длительные транзакции ни к чему хорошему не приводят.

Так же нужно понимать что при создании подключения к базе данных у вас отжирается определенное кол-во оперативной памяти которое для этого необходимо. Если плодить много подключений - можно сожрать всю память.

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

Если вы не хотите постоянно закрывать подключение и курсоры - вы можете оформить код через конструкцию with. Тогда подключение и курсор будут закрывать автоматически, после выполнения блока кода в конструкции.

Как итог:

1) Закрытие позволяет избежать ошибок, связанных с использование одних подключений в разных потоках.
2) Не расстрачивает вашу оперативную память для создания ненужных подключений.
3) Не забивает базу ненужными подключениями.
4) Позволяет правильно распоряжаться транзакциями, и делать в определенной транзакции только определенные действия, а не пихать все в кучу.
5) Вы не упираетесь в лимит подключений и не получаете ненужных об этом ошибок.
6) Вы не получаете ошибок о том, что одно подключение пытается взаимодействовать с таблицей или данными, созданными в другом подключении, и залоченными или не расшаренными на данный момент.

Comments

[wideShift]

А как долго соединение будет висеть на стороне СУБД если я не закрою соединение у себя в приложении?

[Михаил Трайнин]

wideShift, зависит от того какой таймаут для этого вы выставили. Он может быть как указан явно, так и могут быть использованы дефолтные значения.

[wideShift]

Stormx480, еще пара вопросов:
1) Нужно ли делать commit после выполнения SELECT?
2) Если у меня создано несколько курсоров из одного соединения, то каждый курсор это отдельная транзакция? Когда я приложении создаю и завершаю транзакцию?

[Михаил Трайнин]

wideShift,
1) Зависит от уровня изоляции, установленного в Вашей базе данных. Если вы не знаете стоит делать коммит или нет - то мой ответ: "Да, стоит." Для того что бы не плодить атомарных транзакций существует поле(в некоторых случаях переменная в драйвере) - autocommit. Очень её рекомендую. Но, в дальнейшем, руководствуйтесь правилом: "Коммит всегда лучше, чем его отсутствие"

2) Транзакция создается при создании подключения. А курсор - лишь объект для взаимодействия с базой данных. Если вы в одном подключении наплодите кучу курсоров, и один из них совершит неправильные действия - заблокируется вся транзакция сразу, т.е. все курсоры в ней существующие. Так что плодить большое кол-во непоследовательных действий в рамках одной транзакции - плохой тон.

3) Вы можете завершить транзакцию несколькими способами:

а) Выполнять все действия в рамках конструкции with. Тогда после завершения блока кода, транзакция закроется автоматически.
б) Выполнив conn.commit()
в) Выполнив conn.rollback()
г) Закрыв подключение с помощью conn.close()

[wideShift]

Stormx480, Спасибо за подробные ответы