Merge pull request #4562 from codyrigney92/master

Fixes transaction not getting cleaned up with sqlite Closes #4245

changelog.md

@@ -16,6 +16,7 @@
 - [FIXED] instances returned from `bulkCreate` now has `isNewRecord: false` and should be updateable if using `returning: true` with dialects that support it.
 - [FIXED] instances returned from `bulkCreate` now has `isNewRecord: false` and should be updateable if using `returning: true` with dialects that support it.
 - [FIXED] Find with Include with a where clause generates wrong SQL [#3940](https://github.com/sequelize/sequelize/issues/3940)
 - [FIXED] Find with Include with a where clause generates wrong SQL [#3940](https://github.com/sequelize/sequelize/issues/3940)
 - [FIXED] ON DELETE constraint should default to CASCADE if foreignKey has allowNull: false] [#2831](https://github.com/sequelize/sequelize/issues/2831)
 - [FIXED] ON DELETE constraint should default to CASCADE if foreignKey has allowNull: false] [#2831](https://github.com/sequelize/sequelize/issues/2831)
+- [FIXED] sqlite file handle leak
 
 
 # 3.9.0
 # 3.9.0
 - [ADDED] beforeRestore/afterRestore hooks [#4371](https://github.com/sequelize/sequelize/issues/4371)
 - [ADDED] beforeRestore/afterRestore hooks [#4371](https://github.com/sequelize/sequelize/issues/4371)

lib/transaction.js

@@ -230,9 +230,9 @@ Transaction.prototype.setIsolationLevel = function() {
 };
 };
 
 
 Transaction.prototype.cleanup = function() {
 Transaction.prototype.cleanup = function() {
+  var res = this.sequelize.connectionManager.releaseConnection(this.connection);
   this.connection.uuid = undefined;
   this.connection.uuid = undefined;
-
-  return this.sequelize.connectionManager.releaseConnection(this.connection);
+  return res;
 };
 };
 
 
 Transaction.prototype.$clearCls = function () {
 Transaction.prototype.$clearCls = function () {