diff options
| author | Robert Godfrey <rgodfrey@apache.org> | 2015-03-13 23:38:32 +0000 |
|---|---|---|
| committer | Robert Godfrey <rgodfrey@apache.org> | 2015-03-13 23:38:32 +0000 |
| commit | 926e70ce9fdb4616cda00ec6f6fc3bbc0124d247 (patch) | |
| tree | 50ac95bdc90d98ad0f3b0696fc253b4ed67e8798 /qpid/java | |
| parent | ea42a108d6a5ecfdca0da86084efab2065494b23 (diff) | |
| download | qpid-python-926e70ce9fdb4616cda00ec6f6fc3bbc0124d247.tar.gz | |
QPID-6450 : [Java Broker] Allow separate model configuration threads for each virtual host node
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1666609 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java')
46 files changed, 192 insertions, 25 deletions
diff --git a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhost/berkeleydb/BDBVirtualHostImplTest.java b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhost/berkeleydb/BDBVirtualHostImplTest.java index 92da465dbd..24a2ddb071 100644 --- a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhost/berkeleydb/BDBVirtualHostImplTest.java +++ b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhost/berkeleydb/BDBVirtualHostImplTest.java @@ -53,6 +53,8 @@ public class BDBVirtualHostImplTest extends QpidTestCase TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); when(broker.getTaskExecutor()).thenReturn(taskExecutor); + when(broker.getChildExecutor()).thenReturn(taskExecutor); + _storePath = TestFileUtils.createTestDirectory(); @@ -60,6 +62,7 @@ public class BDBVirtualHostImplTest extends QpidTestCase when(_node.getParent(Broker.class)).thenReturn(broker); when(_node.getModel()).thenReturn(BrokerModel.getInstance()); when(_node.getTaskExecutor()).thenReturn(taskExecutor); + when(_node.getChildExecutor()).thenReturn(taskExecutor); when(_node.getConfigurationStore()).thenReturn(mock(DurableConfigurationStore.class)); when(_node.getId()).thenReturn(UUID.randomUUID()); } diff --git a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHARemoteReplicationNodeTest.java b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHARemoteReplicationNodeTest.java index cb9d02c767..4fe2bdc97f 100644 --- a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHARemoteReplicationNodeTest.java +++ b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHARemoteReplicationNodeTest.java @@ -63,6 +63,7 @@ public class BDBHARemoteReplicationNodeTest extends QpidTestCase _taskExecutor = new CurrentThreadTaskExecutor(); _taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); _virtualHostNode = mock(BDBHAVirtualHostNode.class); _configStore = mock(DurableConfigurationStore.class); @@ -74,6 +75,8 @@ public class BDBHARemoteReplicationNodeTest extends QpidTestCase ConfiguredObjectFactory objectFactory = _broker.getObjectFactory(); when(_virtualHostNode.getModel()).thenReturn(objectFactory.getModel()); when(_virtualHostNode.getTaskExecutor()).thenReturn(_taskExecutor); + when(_virtualHostNode.getChildExecutor()).thenReturn(_taskExecutor); + } public void testUpdateRole() @@ -154,4 +157,4 @@ public class BDBHARemoteReplicationNodeTest extends QpidTestCase } -}
\ No newline at end of file +} diff --git a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHAVirtualHostNodeTestHelper.java b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHAVirtualHostNodeTestHelper.java index 0dc72acdb9..9d0e905fc5 100644 --- a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHAVirtualHostNodeTestHelper.java +++ b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBHAVirtualHostNodeTestHelper.java @@ -79,6 +79,8 @@ public class BDBHAVirtualHostNodeTestHelper _taskExecutor = new TaskExecutorImpl(); _taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); + _bdbStorePath = new File(QpidTestCase.TMP_FOLDER, _testName + "." + System.currentTimeMillis()); _bdbStorePath.deleteOnExit(); diff --git a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBVirtualHostNodeTest.java b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBVirtualHostNodeTest.java index 6608312088..812d9a3b19 100644 --- a/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBVirtualHostNodeTest.java +++ b/qpid/java/bdbstore/src/test/java/org/apache/qpid/server/virtualhostnode/berkeleydb/BDBVirtualHostNodeTest.java @@ -29,6 +29,7 @@ import java.util.UUID; import org.apache.qpid.server.configuration.IllegalConfigurationException; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.model.Broker; import org.apache.qpid.server.util.BrokerTestHelper; import org.apache.qpid.test.utils.QpidTestCase; @@ -45,7 +46,9 @@ public class BDBVirtualHostNodeTest extends QpidTestCase { super.setUp(); _broker = BrokerTestHelper.createBrokerMock(); - when(_broker.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(_broker.getTaskExecutor()).thenReturn(taskExecutor); + when(_broker.getChildExecutor()).thenReturn(taskExecutor); _storePath = TestFileUtils.createTestDirectory(); } diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutor.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutor.java index 8d572189b3..71c8c5c058 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutor.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutor.java @@ -26,6 +26,12 @@ import java.util.concurrent.Future; public interface TaskExecutor { + interface Factory + { + TaskExecutor newInstance(); + TaskExecutor newInstance(String name); + } + boolean isRunning(); void start(); @@ -47,4 +53,6 @@ public interface TaskExecutor boolean isTaskExecutorThread(); Executor getExecutor(); + + Factory getFactory(); } diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutorImpl.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutorImpl.java index 5c10d6f7ac..8d1b846e13 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutorImpl.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/configuration/updater/TaskExecutorImpl.java @@ -51,7 +51,17 @@ public class TaskExecutorImpl implements TaskExecutor private final AtomicBoolean _running = new AtomicBoolean(); private volatile ExecutorService _executor; private final ImmediateIfSameThreadExecutor _wrappedExecutor = new ImmediateIfSameThreadExecutor(); + private final String _name; + public TaskExecutorImpl() + { + this(TASK_EXECUTION_THREAD_NAME); + } + + public TaskExecutorImpl(final String name) + { + _name = name; + } @Override public boolean isRunning() @@ -70,7 +80,7 @@ public class TaskExecutorImpl implements TaskExecutor @Override public Thread newThread(Runnable r) { - _taskThread = new TaskThread(r, TASK_EXECUTION_THREAD_NAME, TaskExecutorImpl.this); + _taskThread = new TaskThread(r, _name, TaskExecutorImpl.this); return _taskThread; } }); @@ -419,4 +429,23 @@ public class TaskExecutorImpl implements TaskExecutor return _taskExecutor; } } + + @Override + public Factory getFactory() + { + return new Factory() + { + @Override + public TaskExecutor newInstance() + { + return new TaskExecutorImpl(); + } + + @Override + public TaskExecutor newInstance(final String name) + { + return new TaskExecutorImpl(name); + } + }; + } } diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AbstractConfiguredObject.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AbstractConfiguredObject.java index f9d51028f5..abdd79dc82 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AbstractConfiguredObject.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AbstractConfiguredObject.java @@ -196,7 +196,7 @@ public abstract class AbstractConfiguredObject<X extends ConfiguredObject<X>> im protected AbstractConfiguredObject(final Map<Class<? extends ConfiguredObject>, ConfiguredObject<?>> parents, Map<String, Object> attributes) { - this(parents, attributes, parents.values().iterator().next().getTaskExecutor()); + this(parents, attributes, parents.values().iterator().next().getChildExecutor()); } @@ -1829,6 +1829,12 @@ public abstract class AbstractConfiguredObject<X extends ConfiguredObject<X>> im return _taskExecutor; } + @Override + public TaskExecutor getChildExecutor() + { + return getTaskExecutor(); + } + protected final <C> C runTask(Task<C> task) { return _taskExecutor.run(task); diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Broker.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Broker.java index 6435124bd8..2a6c87c140 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Broker.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Broker.java @@ -197,8 +197,6 @@ public interface Broker<X extends Broker<X>> extends ConfiguredObject<X>, EventL Collection<TrustStore<?>> getTrustStores(); - TaskExecutor getTaskExecutor(); - boolean isManagementMode(); AuthenticationProvider<?> getAuthenticationProvider(SocketAddress localAddress); diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObject.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObject.java index d2ab317f0e..b058ec95b8 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObject.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObject.java @@ -261,6 +261,8 @@ public interface ConfiguredObject<X extends ConfiguredObject<X>> TaskExecutor getTaskExecutor(); + TaskExecutor getChildExecutor(); + ConfiguredObjectFactory getObjectFactory(); Model getModel(); diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/VirtualHost.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/VirtualHost.java index c2338c08d8..24e8900b9e 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/VirtualHost.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/VirtualHost.java @@ -166,9 +166,6 @@ public interface VirtualHost<X extends VirtualHost<X, Q, E>, Q extends Queue<?>, void executeTransaction(TransactionalOperation op); - // TODO - remove this - TaskExecutor getTaskExecutor(); - E getExchange(UUID id); MessageStore getMessageStore(); diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostnode/AbstractVirtualHostNode.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostnode/AbstractVirtualHostNode.java index ebf09b46c3..104d6389ca 100644 --- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostnode/AbstractVirtualHostNode.java +++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostnode/AbstractVirtualHostNode.java @@ -46,6 +46,7 @@ import org.slf4j.LoggerFactory; import org.apache.qpid.exchange.ExchangeDefaults; import org.apache.qpid.server.configuration.IllegalConfigurationException; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.logging.messages.ConfigStoreMessages; import org.apache.qpid.server.logging.subjects.MessageStoreLogSubject; @@ -91,6 +92,7 @@ public abstract class AbstractVirtualHostNode<X extends AbstractVirtualHostNode< private MessageStoreLogSubject _configurationStoreLogSubject; + private TaskExecutor _virtualHostExecutor; @ManagedAttributeField private String _virtualHostInitialConfiguration; @@ -98,6 +100,8 @@ public abstract class AbstractVirtualHostNode<X extends AbstractVirtualHostNode< { super(Collections.<Class<? extends ConfiguredObject>,ConfiguredObject<?>>singletonMap(Broker.class, parent), attributes); + _virtualHostExecutor = getTaskExecutor().getFactory().newInstance("VirtualHostNode-"+getName()+"-Configuration-Thread"); + _virtualHostExecutor.start(); _broker = parent; SystemConfig<?> systemConfig = _broker.getParent(SystemConfig.class); _eventLogger = systemConfig.getEventLogger(); @@ -112,6 +116,12 @@ public abstract class AbstractVirtualHostNode<X extends AbstractVirtualHostNode< } @Override + public TaskExecutor getChildExecutor() + { + return _virtualHostExecutor; + } + + @Override public LifetimePolicy getLifetimePolicy() { return LifetimePolicy.PERMANENT; @@ -309,6 +319,7 @@ public abstract class AbstractVirtualHostNode<X extends AbstractVirtualHostNode< protected void onClose() { closeConfigurationStore(); + _virtualHostExecutor.stop(); } private void closeConfigurationStore() diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/binding/BindingImplTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/binding/BindingImplTest.java index d004f16466..be2b35eaa6 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/binding/BindingImplTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/binding/BindingImplTest.java @@ -63,10 +63,12 @@ public class BindingImplTest extends QpidTestCase SecurityManager securityManager = mock(SecurityManager.class); when(vhost.getSecurityManager()).thenReturn(securityManager); when(queue.getTaskExecutor()).thenReturn(_taskExecutor); + when(queue.getChildExecutor()).thenReturn(_taskExecutor); when(queue.getVirtualHost()).thenReturn(vhost); when(queue.getModel()).thenReturn(_model); ExchangeImpl exchange = mock(ExchangeImpl.class); when(exchange.getTaskExecutor()).thenReturn(_taskExecutor); + when(exchange.getChildExecutor()).thenReturn(_taskExecutor); when(exchange.getModel()).thenReturn(_model); BindingImpl binding = new BindingImpl(attributes, queue, exchange); try diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/PreferencesProviderCreationTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/PreferencesProviderCreationTest.java index 9c85ffa6d9..58c05e337c 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/PreferencesProviderCreationTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/PreferencesProviderCreationTest.java @@ -30,6 +30,7 @@ import java.util.Map; import java.util.UUID; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.model.AuthenticationProvider; import org.apache.qpid.server.model.Broker; import org.apache.qpid.server.model.ConfiguredObjectFactory; @@ -75,7 +76,10 @@ public class PreferencesProviderCreationTest extends QpidTestCase when(_authenticationProvider.getParent(Broker.class)).thenReturn(_broker); when(_authenticationProvider.getObjectFactory()).thenReturn(factory); when(_authenticationProvider.getModel()).thenReturn(factory.getModel()); - when(_authenticationProvider.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(_authenticationProvider.getTaskExecutor()).thenReturn(taskExecutor); + when(_authenticationProvider.getChildExecutor()).thenReturn(taskExecutor); + } public void tearDown() throws Exception diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/VirtualHostCreationTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/VirtualHostCreationTest.java index f732a01c1e..012479dfe0 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/VirtualHostCreationTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/startup/VirtualHostCreationTest.java @@ -64,6 +64,7 @@ public class VirtualHostCreationTest extends TestCase when(systemConfig.getModel()).thenReturn(objectFactory.getModel()); when(systemConfig.getEventLogger()).thenReturn(eventLogger); when(systemConfig.getTaskExecutor()).thenReturn(executor); + when(systemConfig.getChildExecutor()).thenReturn(executor); Broker broker = mock(Broker.class); when(broker.getObjectFactory()).thenReturn(objectFactory); @@ -72,6 +73,7 @@ public class VirtualHostCreationTest extends TestCase when(broker.getCategoryClass()).thenReturn(Broker.class); when(broker.getParent(eq(SystemConfig.class))).thenReturn(systemConfig); when(broker.getTaskExecutor()).thenReturn(executor); + when(broker.getChildExecutor()).thenReturn(executor); _virtualHostNode = mock(VirtualHostNode.class); when(_virtualHostNode.getParent(Broker.class)).thenReturn(broker); @@ -80,6 +82,7 @@ public class VirtualHostCreationTest extends TestCase when(_virtualHostNode.getModel()).thenReturn(objectFactory.getModel()); when(_virtualHostNode.getCategoryClass()).thenReturn(VirtualHostNode.class); when(_virtualHostNode.getTaskExecutor()).thenReturn(executor); + when(_virtualHostNode.getChildExecutor()).thenReturn(executor); } public void testCreateVirtualHostFromStoreConfigAttributes() diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/updater/CurrentThreadTaskExecutor.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/updater/CurrentThreadTaskExecutor.java index 26645722c9..05b6c38011 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/updater/CurrentThreadTaskExecutor.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/configuration/updater/CurrentThreadTaskExecutor.java @@ -158,4 +158,23 @@ public class CurrentThreadTaskExecutor implements TaskExecutor { return MoreExecutors.sameThreadExecutor(); } + + @Override + public Factory getFactory() + { + return new Factory() + { + @Override + public TaskExecutor newInstance() + { + return CurrentThreadTaskExecutor.this; + } + + @Override + public TaskExecutor newInstance(final String name) + { + return CurrentThreadTaskExecutor.this; + } + }; + } } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/consumer/MockConsumer.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/consumer/MockConsumer.java index d625fcba75..3723d19bfe 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/consumer/MockConsumer.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/consumer/MockConsumer.java @@ -35,6 +35,7 @@ import java.util.concurrent.locks.ReentrantLock; import org.apache.qpid.protocol.AMQConstant; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.filter.FilterManager; import org.apache.qpid.server.filter.Filterable; import org.apache.qpid.server.filter.MessageFilter; @@ -310,7 +311,9 @@ public class MockConsumer implements ConsumerTarget ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(_modelObject.getObjectFactory()).thenReturn(factory); when(_modelObject.getModel()).thenReturn(factory.getModel()); - when(_modelObject.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(_modelObject.getTaskExecutor()).thenReturn(taskExecutor); + when(_modelObject.getChildExecutor()).thenReturn(taskExecutor); } @Override diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/FanoutExchangeTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/FanoutExchangeTest.java index 3e0ba31b08..a66759f81c 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/FanoutExchangeTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/FanoutExchangeTest.java @@ -84,6 +84,7 @@ public class FanoutExchangeTest extends TestCase when(_virtualHost.getSecurityManager()).thenReturn(securityManager); when(_virtualHost.getEventLogger()).thenReturn(new EventLogger()); when(_virtualHost.getTaskExecutor()).thenReturn(_taskExecutor); + when(_virtualHost.getChildExecutor()).thenReturn(_taskExecutor); when(_virtualHost.getModel()).thenReturn(BrokerModel.getInstance()); when(_virtualHost.getParent(VirtualHostNode.class)).thenReturn(virtualHostNode); when(_virtualHost.getCategoryClass()).thenReturn(VirtualHost.class); @@ -149,7 +150,9 @@ public class FanoutExchangeTest extends TestCase when(queue.getVirtualHost()).thenReturn(_virtualHost); when(queue.getCategoryClass()).thenReturn(Queue.class); when(queue.getModel()).thenReturn(BrokerModel.getInstance()); - when(queue.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(queue.getTaskExecutor()).thenReturn(taskExecutor); + when(queue.getChildExecutor()).thenReturn(taskExecutor); when(queue.getParent(VirtualHost.class)).thenReturn(_virtualHost); return queue; } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersBindingTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersBindingTest.java index b2d35f690f..eab46020ba 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersBindingTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersBindingTest.java @@ -156,15 +156,21 @@ public class HeadersBindingTest extends TestCase when(_queue.getVirtualHost()).thenReturn(vhost); when(_queue.getModel()).thenReturn(BrokerModel.getInstance()); when(_queue.getTaskExecutor()).thenReturn(executor); + when(_queue.getChildExecutor()).thenReturn(executor); + when(vhost.getSecurityManager()).thenReturn(mock(org.apache.qpid.server.security.SecurityManager.class)); final EventLogger eventLogger = new EventLogger(); when(vhost.getEventLogger()).thenReturn(eventLogger); when(vhost.getTaskExecutor()).thenReturn(executor); + when(vhost.getChildExecutor()).thenReturn(executor); + _exchange = mock(ExchangeImpl.class); when(_exchange.getType()).thenReturn(ExchangeDefaults.HEADERS_EXCHANGE_CLASS); when(_exchange.getEventLogger()).thenReturn(eventLogger); when(_exchange.getModel()).thenReturn(BrokerModel.getInstance()); when(_exchange.getTaskExecutor()).thenReturn(executor); + when(_exchange.getChildExecutor()).thenReturn(executor); + } protected String getQueueName() diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersExchangeTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersExchangeTest.java index c5ad190477..983378a01b 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersExchangeTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/exchange/HeadersExchangeTest.java @@ -89,6 +89,8 @@ public class HeadersExchangeTest extends TestCase when(_virtualHost.getEventLogger()).thenReturn(new EventLogger()); when(_virtualHost.getCategoryClass()).thenReturn(VirtualHost.class); when(_virtualHost.getTaskExecutor()).thenReturn(_taskExecutor); + when(_virtualHost.getChildExecutor()).thenReturn(_taskExecutor); + _factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(_virtualHost.getObjectFactory()).thenReturn(_factory); when(_virtualHost.getModel()).thenReturn(_factory.getModel()); @@ -168,7 +170,9 @@ public class HeadersExchangeTest extends TestCase when(q.getCategoryClass()).thenReturn(Queue.class); when(q.getObjectFactory()).thenReturn(_factory); when(q.getModel()).thenReturn(_factory.getModel()); - when(q.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(q.getTaskExecutor()).thenReturn(taskExecutor); + when(q.getChildExecutor()).thenReturn(taskExecutor); return q; } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/VirtualHostTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/VirtualHostTest.java index 4acc925392..7614936b88 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/VirtualHostTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/VirtualHostTest.java @@ -84,6 +84,7 @@ public class VirtualHostTest extends QpidTestCase _taskExecutor = new CurrentThreadTaskExecutor(); _taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); _virtualHostNode = mock(VirtualHostNode.class); when(_virtualHostNode.getParent(Broker.class)).thenReturn(_broker); @@ -101,6 +102,8 @@ public class VirtualHostTest extends QpidTestCase ConfiguredObjectFactory objectFactory = _broker.getObjectFactory(); when(_virtualHostNode.getModel()).thenReturn(objectFactory.getModel()); when(_virtualHostNode.getTaskExecutor()).thenReturn(_taskExecutor); + when(_virtualHostNode.getChildExecutor()).thenReturn(_taskExecutor); + } @Override diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileBasedGroupProviderImplTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileBasedGroupProviderImplTest.java index 50a4e5a86e..81dba681c0 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileBasedGroupProviderImplTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileBasedGroupProviderImplTest.java @@ -51,6 +51,7 @@ public class FileBasedGroupProviderImplTest extends QpidTestCase _broker = mock(Broker.class); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(BrokerModel.getInstance()); when(_broker.getId()).thenReturn(UUID.randomUUID()); when(_broker.getSecurityManager()).thenReturn(new SecurityManager(_broker, false)); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileSystemPreferencesProviderTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileSystemPreferencesProviderTest.java index f532a9325b..b0a31207a8 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileSystemPreferencesProviderTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/FileSystemPreferencesProviderTest.java @@ -67,6 +67,7 @@ public class FileSystemPreferencesProviderTest extends QpidTestCase _taskExecutor = new CurrentThreadTaskExecutor(); _taskExecutor.start(); when(_authenticationProvider.getTaskExecutor()).thenReturn(_taskExecutor); + when(_authenticationProvider.getChildExecutor()).thenReturn(_taskExecutor); when(_authenticationProvider.getParent(Broker.class)).thenReturn(_broker); ConfiguredObjectFactory objectFactory = _broker.getObjectFactory(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/PortFactoryTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/PortFactoryTest.java index 642ea06ede..b763568ab5 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/PortFactoryTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/adapter/PortFactoryTest.java @@ -96,6 +96,7 @@ public class PortFactoryTest extends QpidTestCase for(ConfiguredObject obj : new ConfiguredObject[]{_authProvider, _broker, _keyStore, _trustStore}) { when(obj.getTaskExecutor()).thenReturn(executor); + when(obj.getChildExecutor()).thenReturn(executor); } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/port/AmqpPortImplTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/port/AmqpPortImplTest.java index 4df5fb50fc..3ef81e7635 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/port/AmqpPortImplTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/model/port/AmqpPortImplTest.java @@ -64,6 +64,7 @@ public class AmqpPortImplTest extends QpidTestCase _broker = mock(Broker.class); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(model); when(_broker.getId()).thenReturn(UUID.randomUUID()); when(_broker.getSecurityManager()).thenReturn(new SecurityManager(_broker, false)); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/LastValueQueueListTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/LastValueQueueListTest.java index 2427470707..faa2d3ffb8 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/LastValueQueueListTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/LastValueQueueListTest.java @@ -30,6 +30,7 @@ import java.util.UUID; import junit.framework.TestCase; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.AMQMessageHeader; import org.apache.qpid.server.message.MessageReference; @@ -67,7 +68,9 @@ public class LastValueQueueListTest extends TestCase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); _queue = new LastValueQueueImpl(queueAttributes, virtualHost); _queue.open(); _list = _queue.getEntries(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/PriorityQueueListTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/PriorityQueueListTest.java index 631731ecc0..49e333c364 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/PriorityQueueListTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/PriorityQueueListTest.java @@ -29,6 +29,7 @@ import java.util.Map; import java.util.UUID; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.AMQMessageHeader; import org.apache.qpid.server.message.MessageReference; @@ -67,7 +68,9 @@ public class PriorityQueueListTest extends QpidTestCase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); PriorityQueueImpl queue = new PriorityQueueImpl(queueAttributes, virtualHost); queue.open(); _list = queue.getEntries(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/QueueEntryImplTestBase.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/QueueEntryImplTestBase.java index 2101a2297f..2eb083bdb3 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/QueueEntryImplTestBase.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/QueueEntryImplTestBase.java @@ -30,6 +30,7 @@ import java.util.UUID; import junit.framework.TestCase; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.MessageInstance; import org.apache.qpid.server.message.MessageInstance.EntryState; @@ -245,7 +246,9 @@ public abstract class QueueEntryImplTestBase extends TestCase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); StandardQueueImpl queue = new StandardQueueImpl(queueAttributes, virtualHost); queue.open(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SimpleQueueEntryImplTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SimpleQueueEntryImplTest.java index a27db98400..f7e9768083 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SimpleQueueEntryImplTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SimpleQueueEntryImplTest.java @@ -29,6 +29,7 @@ import java.util.Map; import java.util.UUID; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.MessageReference; import org.apache.qpid.server.message.ServerMessage; @@ -56,7 +57,9 @@ public class SimpleQueueEntryImplTest extends QueueEntryImplTestBase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); StandardQueueImpl queue = new StandardQueueImpl(queueAttributes, virtualHost); queue.open(); queueEntryList = queue.getEntries(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryListTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryListTest.java index 0c7f19bbd5..8dbe22f5b6 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryListTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryListTest.java @@ -31,6 +31,7 @@ import java.util.Map; import java.util.UUID; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.AMQMessageHeader; import org.apache.qpid.server.message.MessageReference; @@ -95,7 +96,9 @@ public class SortedQueueEntryListTest extends QueueEntryListTestBase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); _testQueue = new SortedQueueImpl(attributes, virtualHost) { SelfValidatingSortedQueueEntryList _entries; diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryTest.java index f88ce5f5f9..0b72af17bc 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/SortedQueueEntryTest.java @@ -30,6 +30,7 @@ import java.util.Map; import java.util.UUID; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.AMQMessageHeader; import org.apache.qpid.server.message.MessageReference; @@ -65,7 +66,9 @@ public class SortedQueueEntryTest extends QueueEntryImplTestBase ConfiguredObjectFactory factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(factory); when(virtualHost.getModel()).thenReturn(factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); SortedQueueImpl queue = new SortedQueueImpl(attributes, virtualHost) { SelfValidatingSortedQueueEntryList _entries; diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/StandardQueueEntryListTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/StandardQueueEntryListTest.java index 73d14a843f..c592622877 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/StandardQueueEntryListTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/queue/StandardQueueEntryListTest.java @@ -31,6 +31,7 @@ import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentMap; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.message.MessageReference; import org.apache.qpid.server.message.ServerMessage; @@ -65,7 +66,9 @@ public class StandardQueueEntryListTest extends QueueEntryListTestBase _factory = new ConfiguredObjectFactoryImpl(BrokerModel.getInstance()); when(virtualHost.getObjectFactory()).thenReturn(_factory); when(virtualHost.getModel()).thenReturn(_factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); _testQueue = new StandardQueueImpl(queueAttributes, virtualHost); _testQueue.open(); _sqel = _testQueue.getEntries(); @@ -115,7 +118,9 @@ public class StandardQueueEntryListTest extends QueueEntryListTestBase when(virtualHost.getEventLogger()).thenReturn(new EventLogger()); when(virtualHost.getObjectFactory()).thenReturn(_factory); when(virtualHost.getModel()).thenReturn(_factory.getModel()); - when(virtualHost.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(virtualHost.getTaskExecutor()).thenReturn(taskExecutor); + when(virtualHost.getChildExecutor()).thenReturn(taskExecutor); StandardQueueImpl queue = new StandardQueueImpl(queueAttributes, virtualHost); queue.open(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileKeyStoreTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileKeyStoreTest.java index c691c21ce2..df1dc82d14 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileKeyStoreTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileKeyStoreTest.java @@ -61,6 +61,7 @@ public class FileKeyStoreTest extends QpidTestCase super.setUp(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(_model); when(_broker.getSecurityManager()).thenReturn(_securityManager); when(_broker.getCategoryClass()).thenReturn(Broker.class); @@ -316,4 +317,4 @@ public class FileKeyStoreTest extends QpidTestCase byte[] fileAsBytes = FileUtils.readFileAsBytes(filename); return DataUrlUtils.getDataUrlForBytes(fileAsBytes); } -}
\ No newline at end of file +} diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileTrustStoreTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileTrustStoreTest.java index 8e7f004923..44577ee018 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileTrustStoreTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/FileTrustStoreTest.java @@ -64,6 +64,8 @@ public class FileTrustStoreTest extends QpidTestCase super.setUp(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); + when(_broker.getModel()).thenReturn(_model); when(_broker.getSecurityManager()).thenReturn(_securityManager); when(_broker.getCategoryClass()).thenReturn(Broker.class); @@ -294,4 +296,4 @@ public class FileTrustStoreTest extends QpidTestCase byte[] fileAsBytes = FileUtils.readFileAsBytes(filename); return DataUrlUtils.getDataUrlForBytes(fileAsBytes); } -}
\ No newline at end of file +} diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaKeyStoreTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaKeyStoreTest.java index f8d4ee8d6c..ff33ed1514 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaKeyStoreTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaKeyStoreTest.java @@ -63,6 +63,7 @@ public class NonJavaKeyStoreTest extends QpidTestCase super.setUp(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(_model); when(_broker.getSecurityManager()).thenReturn(_securityManager); _testResources = new ArrayList<>(); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaTrustStoreTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaTrustStoreTest.java index 798d9fcbe5..13609ff868 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaTrustStoreTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/NonJavaTrustStoreTest.java @@ -51,6 +51,7 @@ public class NonJavaTrustStoreTest extends QpidTestCase super.setUp(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(_model); when(_broker.getSecurityManager()).thenReturn(_securityManager); } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/auth/manager/ManagedAuthenticationManagerTestBase.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/auth/manager/ManagedAuthenticationManagerTestBase.java index dd92d3ebca..65cf7ad9d9 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/auth/manager/ManagedAuthenticationManagerTestBase.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/security/auth/manager/ManagedAuthenticationManagerTestBase.java @@ -61,6 +61,7 @@ abstract class ManagedAuthenticationManagerTestBase extends QpidTestCase _broker = BrokerTestHelper.createBrokerMock(); _securityManager = mock(SecurityManager.class); when(_broker.getTaskExecutor()).thenReturn(_executor); + when(_broker.getChildExecutor()).thenReturn(_executor); when(_broker.getSecurityManager()).thenReturn(_securityManager); final Map<String, Object> attributesMap = new HashMap<String, Object>(); attributesMap.put(AuthenticationProvider.NAME, getTestName()); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/AbstractDurableConfigurationStoreTestCase.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/AbstractDurableConfigurationStoreTestCase.java index 7aaa437723..dbdf606fa6 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/AbstractDurableConfigurationStoreTestCase.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/AbstractDurableConfigurationStoreTestCase.java @@ -42,6 +42,7 @@ import org.mockito.stubbing.Answer; import org.apache.qpid.common.AMQPFilterTypes; import org.apache.qpid.server.binding.BindingImpl; import org.apache.qpid.server.configuration.updater.CurrentThreadTaskExecutor; +import org.apache.qpid.server.configuration.updater.TaskExecutor; import org.apache.qpid.server.exchange.ExchangeImpl; import org.apache.qpid.server.logging.EventLogger; import org.apache.qpid.server.model.Binding; @@ -456,7 +457,9 @@ public abstract class AbstractDurableConfigurationStoreTestCase extends QpidTest when(queue.getAlternateExchange()).thenReturn(alternateExchange); when(queue.getCategoryClass()).thenReturn((Class)Queue.class); when(queue.isDurable()).thenReturn(true); - when(queue.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(queue.getTaskExecutor()).thenReturn(taskExecutor); + when(queue.getChildExecutor()).thenReturn(taskExecutor); final VirtualHostImpl vh = mock(VirtualHostImpl.class); when(vh.getSecurityManager()).thenReturn(mock(SecurityManager.class)); @@ -517,7 +520,9 @@ public abstract class AbstractDurableConfigurationStoreTestCase extends QpidTest when(exchange.isDurable()).thenReturn(true); when(exchange.getObjectFactory()).thenReturn(_factory); when(exchange.getModel()).thenReturn(_factory.getModel()); - when(exchange.getTaskExecutor()).thenReturn(CurrentThreadTaskExecutor.newStartedInstance()); + TaskExecutor taskExecutor = CurrentThreadTaskExecutor.newStartedInstance(); + when(exchange.getTaskExecutor()).thenReturn(taskExecutor); + when(exchange.getChildExecutor()).thenReturn(taskExecutor); ConfiguredObjectRecord exchangeRecord = mock(ConfiguredObjectRecord.class); when(exchangeRecord.getId()).thenReturn(_exchangeId); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/VirtualHostStoreUpgraderAndRecovererTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/VirtualHostStoreUpgraderAndRecovererTest.java index 0e82eb6c73..2b65d30fe4 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/VirtualHostStoreUpgraderAndRecovererTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/store/VirtualHostStoreUpgraderAndRecovererTest.java @@ -88,6 +88,7 @@ public class VirtualHostStoreUpgraderAndRecovererTest extends QpidTestCase Broker<?> broker = mock(Broker.class); when(broker.getParent(SystemConfig.class)).thenReturn(systemConfig); when(broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(broker.getChildExecutor()).thenReturn(_taskExecutor); when(broker.getModel()).thenReturn(BrokerModel.getInstance()); _durableConfigurationStore = mock(DurableConfigurationStore.class); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/util/BrokerTestHelper.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/util/BrokerTestHelper.java index 0f256d0999..15be5c0282 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/util/BrokerTestHelper.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/util/BrokerTestHelper.java @@ -101,6 +101,9 @@ public class BrokerTestHelper when(broker.getTaskExecutor()).thenReturn(TASK_EXECUTOR); when(systemConfig.getTaskExecutor()).thenReturn(TASK_EXECUTOR); + when(broker.getChildExecutor()).thenReturn(TASK_EXECUTOR); + when(systemConfig.getChildExecutor()).thenReturn(TASK_EXECUTOR); + return broker; } @@ -127,6 +130,7 @@ public class BrokerTestHelper VirtualHostNode virtualHostNode = mock(VirtualHostNode.class); when(virtualHostNode.getName()).thenReturn((String) attributes.get(VirtualHostNode.NAME)); when(virtualHostNode.getTaskExecutor()).thenReturn(TASK_EXECUTOR); + when(virtualHostNode.getChildExecutor()).thenReturn(TASK_EXECUTOR); when(virtualHostNode.getParent(eq(Broker.class))).thenReturn(broker); @@ -142,6 +146,8 @@ public class BrokerTestHelper when(virtualHostNode.getObjectFactory()).thenReturn(objectFactory); when(virtualHostNode.getCategoryClass()).thenReturn(VirtualHostNode.class); when(virtualHostNode.getTaskExecutor()).thenReturn(TASK_EXECUTOR); + when(virtualHostNode.getChildExecutor()).thenReturn(TASK_EXECUTOR); + AbstractVirtualHost host = (AbstractVirtualHost) objectFactory.create(VirtualHost.class, attributes, virtualHostNode ); host.start(); @@ -209,6 +215,7 @@ public class BrokerTestHelper when(virtualHost.getObjectFactory()).thenReturn(objectFactory); when(virtualHost.getModel()).thenReturn(objectFactory.getModel()); when(virtualHost.getTaskExecutor()).thenReturn(TASK_EXECUTOR); + when(virtualHost.getChildExecutor()).thenReturn(TASK_EXECUTOR); final Map<String,Object> attributes = new HashMap<String, Object>(); attributes.put(org.apache.qpid.server.model.Exchange.ID, UUIDGenerator.generateExchangeUUID("amp.direct", virtualHost.getName())); attributes.put(org.apache.qpid.server.model.Exchange.NAME, "amq.direct"); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/AbstractVirtualHostTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/AbstractVirtualHostTest.java index 5a99538edc..47fd5a7398 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/AbstractVirtualHostTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/AbstractVirtualHostTest.java @@ -68,11 +68,13 @@ public class AbstractVirtualHostTest extends QpidTestCase _taskExecutor = new TaskExecutorImpl(); _taskExecutor.start(); when(broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(broker.getChildExecutor()).thenReturn(_taskExecutor); _node = mock(VirtualHostNode.class); when(_node.getParent(Broker.class)).thenReturn(broker); when(_node.getModel()).thenReturn(BrokerModel.getInstance()); when(_node.getTaskExecutor()).thenReturn(_taskExecutor); + when(_node.getChildExecutor()).thenReturn(_taskExecutor); when(_node.getConfigurationStore()).thenReturn(mock(DurableConfigurationStore.class)); when(_node.getCategoryClass()).thenReturn(VirtualHostNode.class); diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/VirtualHostQueueCreationTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/VirtualHostQueueCreationTest.java index fd6748f500..ad02208c6b 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/VirtualHostQueueCreationTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhost/VirtualHostQueueCreationTest.java @@ -83,6 +83,7 @@ public class VirtualHostQueueCreationTest extends QpidTestCase when(broker.getSecurityManager()).thenReturn(securityManager); when(broker.getModel()).thenReturn(objectFactory.getModel()); when(broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(broker.getChildExecutor()).thenReturn(_taskExecutor); _virtualHostNode = mock(VirtualHostNode.class); when(_virtualHostNode.getParent(Broker.class)).thenReturn(broker); @@ -90,6 +91,7 @@ public class VirtualHostQueueCreationTest extends QpidTestCase when(_virtualHostNode.getObjectFactory()).thenReturn(objectFactory); when(_virtualHostNode.getModel()).thenReturn(objectFactory.getModel()); when(_virtualHostNode.getTaskExecutor()).thenReturn(_taskExecutor); + when(_virtualHostNode.getChildExecutor()).thenReturn(_taskExecutor); when(((VirtualHostNode)_virtualHostNode).getCategoryClass()).thenReturn(VirtualHostNode.class); _virtualHost = createHost(); } diff --git a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhostnode/AbstractStandardVirtualHostNodeTest.java b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhostnode/AbstractStandardVirtualHostNodeTest.java index deb93d6527..c188163904 100644 --- a/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhostnode/AbstractStandardVirtualHostNodeTest.java +++ b/qpid/java/broker-core/src/test/java/org/apache/qpid/server/virtualhostnode/AbstractStandardVirtualHostNodeTest.java @@ -83,6 +83,8 @@ public class AbstractStandardVirtualHostNodeTest extends QpidTestCase _taskExecutor = new CurrentThreadTaskExecutor(); _taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); + } @Override diff --git a/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderFactoryTest.java b/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderFactoryTest.java index 2a691b3652..c51ea64d9c 100644 --- a/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderFactoryTest.java +++ b/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderFactoryTest.java @@ -58,6 +58,8 @@ public class ACLFileAccessControlProviderFactoryTest extends QpidTestCase TaskExecutor taskExecutor = new CurrentThreadTaskExecutor(); taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(taskExecutor); + when(_broker.getChildExecutor()).thenReturn(taskExecutor); + } public void testCreateInstanceWhenAclFileIsNotPresent() diff --git a/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderImplTest.java b/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderImplTest.java index 82c209bb6f..abe8c6fd9f 100644 --- a/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderImplTest.java +++ b/qpid/java/broker-plugins/access-control/src/test/java/org/apache/qpid/server/security/access/plugins/ACLFileAccessControlProviderImplTest.java @@ -52,6 +52,7 @@ public class ACLFileAccessControlProviderImplTest extends QpidTestCase _broker = mock(Broker.class); when(_broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(_broker.getChildExecutor()).thenReturn(_taskExecutor); when(_broker.getModel()).thenReturn(_model); when(_broker.getId()).thenReturn(UUID.randomUUID()); } diff --git a/qpid/java/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/HttpManagementTest.java b/qpid/java/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/HttpManagementTest.java index 4327292336..aa3ebe055d 100644 --- a/qpid/java/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/HttpManagementTest.java +++ b/qpid/java/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/HttpManagementTest.java @@ -62,6 +62,8 @@ public class HttpManagementTest extends QpidTestCase TaskExecutor taskExecutor = new TaskExecutorImpl(); taskExecutor.start(); when(_broker.getTaskExecutor()).thenReturn(taskExecutor); + when(_broker.getChildExecutor()).thenReturn(taskExecutor); + Map<String, Object> attributes = new HashMap<String, Object>(); attributes.put(HttpManagement.HTTP_BASIC_AUTHENTICATION_ENABLED, false); diff --git a/qpid/java/systests/src/test/java/org/apache/qpid/server/store/VirtualHostMessageStoreTest.java b/qpid/java/systests/src/test/java/org/apache/qpid/server/store/VirtualHostMessageStoreTest.java index b243769b32..ab7d84e7fa 100644 --- a/qpid/java/systests/src/test/java/org/apache/qpid/server/store/VirtualHostMessageStoreTest.java +++ b/qpid/java/systests/src/test/java/org/apache/qpid/server/store/VirtualHostMessageStoreTest.java @@ -132,6 +132,7 @@ public class VirtualHostMessageStoreTest extends QpidTestCase _taskExecutor = new TaskExecutorImpl(); _taskExecutor.start(); when(broker.getTaskExecutor()).thenReturn(_taskExecutor); + when(broker.getChildExecutor()).thenReturn(_taskExecutor); ConfiguredObjectFactory factory = broker.getObjectFactory(); Map<String, Object> nodeAttributes = new HashMap<>(); |
