This is an automated email from the ASF dual-hosted git repository.
bschuchardt pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new c87afbe GEODE-5261 - Add a test for connection-pool prefill during
shutdown
c87afbe is described below
commit c87afbe4e952de9317d69ec6b995d7709c95cb5b
Author: Bruce Schuchardt <[email protected]>
AuthorDate: Wed May 30 08:36:56 2018 -0700
GEODE-5261 - Add a test for connection-pool prefill during shutdown
Fixing NPE encountered in a couple of DUnit tests
---
.../org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java
b/geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java
index 51857d3..e3abb31 100644
---
a/geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java
+++
b/geode-core/src/main/java/org/apache/geode/cache/client/internal/ConnectionFactoryImpl.java
@@ -132,7 +132,7 @@ public class ConnectionFactoryImpl implements
ConnectionFactory {
}
testFailedConnectionToServer = true;
} finally {
- if (!initialized) {
+ if (!initialized && connection != null) {
connection.destroy();
failureTracker.addFailure();
connection = null;
--
To stop receiving notification emails like this one, please contact
[email protected].
