tagging release 3.276

dxcity · dxcity · commit c6c2750fb7c2 · 2019-05-28T16:39:19.000+03:00
diff --git a/ReleaseNotes.txt b/ReleaseNotes.txt
@@ -1,4 +1,8 @@
 
+QDS 3.276:
+
+* [QD-1156] RMI: Memory leak of one-way requests on server side
+
 QDS 3.275:
 
 * [QD-1155] dxFeed API: OrderBookModel memory leak
diff --git a/auth/pom.xml b/auth/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-api/pom.xml b/dxfeed-api/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-bin/pom.xml b/dxfeed-bin/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-codegen-verify/pom.xml b/dxfeed-codegen-verify/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-codegen/pom.xml b/dxfeed-codegen/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-impl/pom.xml b/dxfeed-impl/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-ipf-filter/pom.xml b/dxfeed-ipf-filter/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
     </parent>
     <modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-news/pom.xml b/dxfeed-news/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-ondemand/pom.xml b/dxfeed-ondemand/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-plotter/pom.xml b/dxfeed-plotter/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-promise/pom.xml b/dxfeed-promise/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-samples/pom.xml b/dxfeed-samples/pom.xml
@@ -14,13 +14,13 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
 	<artifactId>dxfeed-samples</artifactId>
-  <version>3.275</version>
+  <version>3.276</version>
 	<packaging>jar</packaging>
 
 	<properties>
diff --git a/dxfeed-tools/pom.xml b/dxfeed-tools/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-viewer-installer/pom.xml b/dxfeed-viewer-installer/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-viewer/pom.xml b/dxfeed-viewer/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/dxfeed-webservice-impl/pom.xml b/dxfeed-webservice-impl/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxfeed-webservice/pom.xml b/dxfeed-webservice/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/dxlib-qd-sandbox/pom.xml b/dxlib-qd-sandbox/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/dxlib/pom.xml b/dxlib/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/license/pom.xml b/license/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/mars-sample/pom.xml b/mars-sample/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/mars/pom.xml b/mars/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/pom.xml b/pom.xml
@@ -16,7 +16,7 @@
 	<groupId>com.devexperts.qd</groupId>
 	<artifactId>QD</artifactId>
 	<packaging>pom</packaging>
-	<version>3.275</version>
+	<version>3.276</version>
 
 	<inceptionYear>2002</inceptionYear>
 	<organization>
diff --git a/proto-sample/pom.xml b/proto-sample/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/proto-ssl/pom.xml b/proto-ssl/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/proto/pom.xml b/proto/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/qd-core/pom.xml b/qd-core/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/qd-dataextractor/pom.xml b/qd-dataextractor/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
 
diff --git a/qd-logger/pom.xml b/qd-logger/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/qd-nio/pom.xml b/qd-nio/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/qd-rmi/pom.xml b/qd-rmi/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/qd-rmi/src/main/java/com/devexperts/rmi/impl/MessageProcessor.java b/qd-rmi/src/main/java/com/devexperts/rmi/impl/MessageProcessor.java
@@ -276,7 +276,9 @@ void createAndSubmitTask(RMIChannelImpl channel, ServerRequestInfo requestInfo)
 		if (RMIEndpointImpl.RMI_TRACE_LOG)
 			log.trace("Create task " + task + " at " + connection);
 
-		connection.tasksManager.registerTask(task);
+		if (requestInfo.message.getRequestType() != RMIRequestType.ONE_WAY)
+			connection.tasksManager.registerTask(task);
+
 		Executor executor = service.getExecutor();
 		if (executor == null) {
 			executor = nestedTask ? channel.getExecutor() : connection.endpoint.getServer().getDefaultExecutor();
diff --git a/qd-sample/pom.xml b/qd-sample/pom.xml
@@ -14,7 +14,7 @@
 	<parent>
 		<artifactId>QD</artifactId>
 		<groupId>com.devexperts.qd</groupId>
-		<version>3.275</version>
+		<version>3.276</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 	<modelVersion>4.0.0</modelVersion>
diff --git a/qd-sample/src/test/java/com/devexperts/rmi/test/RMIAsynchronousFunctionalityTest.java b/qd-sample/src/test/java/com/devexperts/rmi/test/RMIAsynchronousFunctionalityTest.java
@@ -12,6 +12,7 @@
 package com.devexperts.rmi.test;
 
 import java.io.IOException;
+import java.lang.ref.WeakReference;
 import java.security.AccessController;
 import java.util.Arrays;
 import java.util.concurrent.*;
@@ -554,8 +555,8 @@ public void testReturnedPromise() throws InterruptedException {
 	@Test
 	public void testOneWaySending() throws InterruptedException {
 		channelLogic.setForward(true);
-		RMIFunctionalityTest.CompletingPing impl = new RMIFunctionalityTest.CompletingPing();
-		NTU.exportServices(remoteEndpoint.getServer(), new RMIServiceImplementation<>(impl, RMICommonTest.Ping.class), channelLogic);
+		RMIFunctionalityTest.CompletingPing pingService = new RMIFunctionalityTest.CompletingPing();
+		NTU.exportServices(remoteEndpoint.getServer(), new RMIServiceImplementation<>(pingService, RMICommonTest.Ping.class), channelLogic);
 		server.getServer().export(privateEndpoint.getClient().getService("*"));
 
 		connectDefault(false, 93, 95);
@@ -569,6 +570,16 @@ public void testOneWaySending() throws InterruptedException {
 		}
 		RMIRequest<Void> request = channelLogic.clientPort.createRequest(new RMIRequestMessage<>(RMIRequestType.ONE_WAY, operation));
 		request.send();
-		impl.waitForCompletion();
+		pingService.waitForCompletion();
+
+		assertEquals(1, pingService.rmiTasks.size());
+		WeakReference<RMITask> taskRef = pingService.rmiTasks.remove();
+		long deadline = System.nanoTime() + TimeUnit.SECONDS.toNanos(10);
+		while (taskRef.get() != null && System.nanoTime() < deadline ) {
+			System.gc();
+			Thread.sleep(100);
+		}
+		assertNull("RMI task leaked", taskRef.get());
 	}
+
 }
diff --git a/qd-sample/src/test/java/com/devexperts/rmi/test/RMIFunctionalityTest.java b/qd-sample/src/test/java/com/devexperts/rmi/test/RMIFunctionalityTest.java
@@ -12,6 +12,7 @@
 package com.devexperts.rmi.test;
 
 import java.io.Serializable;
+import java.lang.ref.WeakReference;
 import java.lang.reflect.UndeclaredThrowableException;
 import java.security.AccessController;
 import java.security.cert.X509Certificate;
@@ -571,10 +572,14 @@ public void testCancelOrAbort() {
 
 
 	public static class CompletingPing implements RMICommonTest.Ping {
+		final Queue<WeakReference<RMITask>> rmiTasks = new ConcurrentLinkedQueue<>();
 		boolean done = false;
 
 		@Override
 		public synchronized void ping() {
+			RMITask task = RMITask.current();
+			if (task != null)
+				rmiTasks.add(new WeakReference<>(task));
 			done = true;
 			notifyAll();
 		}
diff --git a/qd-samplecert/pom.xml b/qd-samplecert/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/qd-servlet/pom.xml b/qd-servlet/pom.xml
@@ -14,7 +14,7 @@
     <parent>
         <artifactId>QD</artifactId>
         <groupId>com.devexperts.qd</groupId>
-        <version>3.275</version>
+        <version>3.276</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <modelVersion>4.0.0</modelVersion>
diff --git a/qd-stripe/pom.xml b/qd-stripe/pom.xml
diff --git a/qd-tools/pom.xml b/qd-tools/pom.xml
diff --git a/qds-file/pom.xml b/qds-file/pom.xml
diff --git a/qds-monitoring/pom.xml b/qds-monitoring/pom.xml
diff --git a/qds-tools/pom.xml b/qds-tools/pom.xml
diff --git a/qds/pom.xml b/qds/pom.xml
diff --git a/rt-api-builder/pom.xml b/rt-api-builder/pom.xml
diff --git a/teamcity-version/pom.xml b/teamcity-version/pom.xml
