id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
17251	select_for_update tests threads should close connections manually	Anssi Kääriäinen	Aymeric Augustin	"This is a small patch which closes two connections taken in threads.

It is good to close the conncetions manually, as they are thread locals in DatabaseWrapper, and thus new connections are created for new threads.

The connections will be closed by GC eventually, but IMHO it is good practice to close the connections manually, as they will not be used again anyways. They will be idle in transaction until they happen to be GCed."	Cleanup/optimization	closed	Uncategorized		Normal	fixed			Accepted	1	0	0	0	0	0