From 5688252257eb5d715100d7313b75fd5741ddf7e1 Mon Sep 17 00:00:00 2001 From: Oleg Pudeyev Date: Mon, 14 Oct 2019 19:36:22 -0400 Subject: [PATCH 1/3] SPEC-1396 Drivers MUST clear connection pools when SDAM monitoring fails due to a network error --- .../server-discovery-and-monitoring.rst | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst index 163a04a574..fca5733e0f 100644 --- a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst +++ b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst @@ -1308,10 +1308,11 @@ Error handling Network error during server check ````````````````````````````````` -When a server `check`_ fails due to a network error, -the client SHOULD clear its connection pool for the server: +When a server `check`_ fails due to a network error (including a timeout), +the client MUST clear its connection pool for the server: if the monitor's socket is bad it is likely that all are. -(See `JAVA-1252 `_.) +(See `JAVA-1252 `_, +`SPEC-1396 `_.) Once a server is connected, the client MUST change its type to Unknown From 926d522e0d9d13c86ff36554c774753981e3e88d Mon Sep 17 00:00:00 2001 From: Oleg Pudeyev Date: Thu, 31 Oct 2019 17:46:04 -0400 Subject: [PATCH 2/3] Use "network timeout" --- .../server-discovery-and-monitoring.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst index fca5733e0f..e9be9f2d5c 100644 --- a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst +++ b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst @@ -1308,7 +1308,7 @@ Error handling Network error during server check ````````````````````````````````` -When a server `check`_ fails due to a network error (including a timeout), +When a server `check`_ fails due to a network error (including a network timeout), the client MUST clear its connection pool for the server: if the monitor's socket is bad it is likely that all are. (See `JAVA-1252 `_, From e8dbd533f0b27379c3e525e263b86b8be28ab25e Mon Sep 17 00:00:00 2001 From: Oleg Pudeyev Date: Thu, 31 Oct 2019 17:46:26 -0400 Subject: [PATCH 3/3] Remove spec ticket reference --- .../server-discovery-and-monitoring.rst | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst index e9be9f2d5c..568856f008 100644 --- a/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst +++ b/source/server-discovery-and-monitoring/server-discovery-and-monitoring.rst @@ -1311,8 +1311,7 @@ Network error during server check When a server `check`_ fails due to a network error (including a network timeout), the client MUST clear its connection pool for the server: if the monitor's socket is bad it is likely that all are. -(See `JAVA-1252 `_, -`SPEC-1396 `_.) +(See `JAVA-1252 `_). Once a server is connected, the client MUST change its type to Unknown