Merge pull request #845 from rmsamitha/5.3.x

Fix continuous broken-pipe error logs in binary transport
wso2 · Oct 11, 2024 · 2191a9a · 2191a9a
2 parents 5f0284b + ce15030
commit 2191a9a
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/...atabridge.agent/src/main/java/org/wso2/carbon/databridge/agent/endpoint/DataEndpoint.java b/...atabridge.agent/src/main/java/org/wso2/carbon/databridge/agent/endpoint/DataEndpoint.java
@@ -73,6 +73,8 @@ public abstract class DataEndpoint {
 
     public static HashMap<String, Long> delayMap=new HashMap<String, Long>();
 
+    public boolean invalidateTransportPool = false;
+
     public long getReConnectTimestamp() {
         return reConnectTimestamp;
     }
@@ -349,6 +351,15 @@ public void setPoolSemaphore(Semaphore semaphore) {
 
         private void publish() throws DataEndpointException, SessionTimeoutException, UndefinedEventTypeException {
             Object client = getClient();
+            if (invalidateTransportPool) {
+                log.debug(
+                        "invalidateTransportPool' is 'true'. Going to discard existing client and get new client " +
+                                "for the DataEndpoint");
+                discardClient(client);
+                client = getClient();
+                invalidateTransportPool = false;
+                log.debug("'invalidateTransportPool' is set to 'false' for the DataEndpoint");
+            }
             try {
                 send(client, this.events);
                 semaphoreRelease();

diff --git a/...nt/src/main/java/org/wso2/carbon/databridge/agent/endpoint/binary/BinaryDataEndpoint.java b/...nt/src/main/java/org/wso2/carbon/databridge/agent/endpoint/binary/BinaryDataEndpoint.java
@@ -17,6 +17,8 @@
 */
 package org.wso2.carbon.databridge.agent.endpoint.binary;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.wso2.carbon.databridge.agent.endpoint.DataEndpoint;
 import org.wso2.carbon.databridge.agent.exception.DataEndpointAuthenticationException;
 import org.wso2.carbon.databridge.agent.exception.DataEndpointException;
@@ -35,6 +37,8 @@
  */
 public class BinaryDataEndpoint extends DataEndpoint {
 
+    private static Log log = LogFactory.getLog(BinaryDataEndpoint.class);
+
     @Override
     protected String login(Object client, String userName, String password)
             throws DataEndpointAuthenticationException, DataEndpointLoginException {
@@ -84,6 +88,8 @@ protected void send(Object client, List<Event> events) throws DataEndpointExcept
             } else if (e instanceof SessionTimeoutException) {
                 throw new SessionTimeoutException("Binary Session Expired Exception ", e);
             } else {
+                log.debug("Setting 'invalidateTransportPool' to 'true' for binary data transport");
+                this.invalidateTransportPool = true;
                 throw new DataEndpointException("Error while trying to publish events to data receiver :"
                         + socket.getRemoteSocketAddress().toString(), e);
             }