Apache
/
activemq
mirror of https://github.com/apache/activemq.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test case that shows something is wrong with start logic on pooled connection factory, the vm test variant was getting serialized on the broker vm transport server

This commit is contained in:
gtully 2014-06-09 14:21:24 +01:00
parent 27b3a7c344
commit 736ffc9b96
1 changed files with 41 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
activemq-jms-pool/src/test/java/org/apache/activemq/jms/pool/PooledConnectionFactoryTest.java
View File

@ -34,9 +34,11 @@ import junit.framework.TestSuite;
import org.apache.activemq.ActiveMQConnection;
import org.apache.activemq.ActiveMQConnectionFactory;
import org.apache.activemq.broker.BrokerService;
import org.apache.activemq.command.ConnectionId;
import org.apache.activemq.util.Wait;
import org.apache.log4j.Logger;
import org.junit.Ignore;
/**
* Checks the behavior of the PooledConnectionFactory when the maximum amount of
@ -203,50 +205,58 @@ public class PooledConnectionFactoryTest extends TestCase {
doTestConcurrentCreateGetsUniqueConnection(false);
}
@Ignore("something up - don't know why the start call to createConnection does not cause close - but that does not fix it either!")
public void testConcurrentCreateGetsUniqueConnectionCreateOnStart() throws Exception {
doTestConcurrentCreateGetsUniqueConnection(true);
}
private void doTestConcurrentCreateGetsUniqueConnection(boolean createOnStart) throws Exception {
final int numConnections = 50;
BrokerService brokerService = new BrokerService();
brokerService.setPersistent(false);
brokerService.addConnector("tcp://localhost:0");
brokerService.start();
final ActiveMQConnectionFactory amq = new ActiveMQConnectionFactory("vm://broker1?marshal=false&broker.persistent=false");
final PooledConnectionFactory cf = new PooledConnectionFactory();
cf.setConnectionFactory(amq);
cf.setMaxConnections(numConnections);
cf.setCreateConnectionOnStartup(createOnStart);
try {
final int numConnections = 2;
final ConcurrentHashMap<ConnectionId, Connection> connections =
new ConcurrentHashMap<ConnectionId, Connection>();
final ExecutorService executor = Executors.newFixedThreadPool(numConnections / 2);
final ActiveMQConnectionFactory amq = new ActiveMQConnectionFactory(brokerService.getTransportConnectors().get(0).getPublishableConnectString());
final PooledConnectionFactory cf = new PooledConnectionFactory();
cf.setConnectionFactory(amq);
cf.setMaxConnections(numConnections);
cf.setCreateConnectionOnStartup(createOnStart);
cf.start();
for (int i = 0; i < numConnections; ++i) {
executor.execute(new Runnable() {
final ConcurrentHashMap<ConnectionId, Connection> connections =
new ConcurrentHashMap<ConnectionId, Connection>();
final ExecutorService executor = Executors.newFixedThreadPool(numConnections);
@Override
public void run() {
try {
PooledConnection pooled = (PooledConnection) cf.createConnection();
ActiveMQConnection amq = (ActiveMQConnection) pooled.getConnection();
connections.put(amq.getConnectionInfo().getConnectionId(), pooled);
} catch (JMSException e) {
for (int i = 0; i < numConnections; ++i) {
executor.execute(new Runnable() {
@Override
public void run() {
try {
PooledConnection pooled = (PooledConnection) cf.createConnection();
ActiveMQConnection amq = (ActiveMQConnection) pooled.getConnection();
connections.put(amq.getConnectionInfo().getConnectionId(), pooled);
} catch (JMSException e) {
}
}
}
});
}
assertTrue("Should have all unique connections", Wait.waitFor(new Wait.Condition() {
@Override
public boolean isSatisified() throws Exception {
return connections.size() == numConnections;
});
}
}));
executor.shutdown();
assertTrue(executor.awaitTermination(5, TimeUnit.SECONDS));
connections.clear();
cf.stop();
executor.shutdown();
assertTrue(executor.awaitTermination(5, TimeUnit.SECONDS));
assertEquals("Should have all unique connections", numConnections, connections.size());
connections.clear();
cf.stop();
} finally {
brokerService.stop();
}
}
/**