Skip to content

Commit 4744c69

Browse files
committed
Revert queueTimeout fix as it requires more analysis
1 parent b87fef5 commit 4744c69

File tree

10 files changed

+316
-350
lines changed

10 files changed

+316
-350
lines changed

doc/src/release_notes.rst

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,6 @@ Common Changes
5858
be raised as an ``NJS-500`` error with the underlying reason being
5959
included in the error message.
6060

61-
#) Improved connection pool's queueTimeout behavior to account for network
62-
delays and pool expansion delays.
63-
6461
#) Fixed bug which does not trigger PL/SQL functions registered for callbacks
6562
properly when using implicit pooling with temporary LOBs creation.
6663

lib/connection.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -797,12 +797,7 @@ class Connection extends EventEmitter {
797797

798798
this._closing = true;
799799
try {
800-
// If connection is part of a pool, notify the pool of its availability
801-
if (this._pool) {
802-
await this._pool._release(this._impl, options);
803-
} else {
804-
await this._impl.close(options);
805-
}
800+
await this._impl.close(options);
806801
} finally {
807802
this._closing = false;
808803
}
@@ -814,6 +809,7 @@ class Connection extends EventEmitter {
814809
}
815810
}
816811
this._dbObjectClasses.clear();
812+
this.emit('_afterConnClose');
817813
}
818814

819815
//---------------------------------------------------------------------------

0 commit comments

Comments
 (0)