diff options
author | Harald Musum <musum@verizonmedia.com> | 2021-04-05 19:00:55 +0200 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2021-04-05 19:00:55 +0200 |
commit | 0c8351ca3e166b69955cdfe8b0221e926ce41147 (patch) | |
tree | 598ce4c886d66f701e69beae96d39b409edf9ecd /configserver/src/test | |
parent | 36438491bdb0bfdd0c9d6e894e6ae99b9468e5ef (diff) |
Use separate executors for ZooKeeper watchers for applications and sessions
Session events might involve doing expensive operations (e.g. creating
config models for an application). Having just one executor has been
observed to lead to events for applications take so long that some
operations (e.g. deleting an application) times out and prevents holding
state in sync across config servers.
Diffstat (limited to 'configserver/src/test')
2 files changed, 2 insertions, 0 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TenantRepositoryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TenantRepositoryTest.java index b3d1495eb6b..6f7e0541cc7 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TenantRepositoryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TenantRepositoryTest.java @@ -212,6 +212,7 @@ public class TenantRepositoryTest { ConfigCurator.create(new MockCurator()), Metrics.createTestMetrics(), new StripedExecutor<>(new InThreadExecutorService()), + new StripedExecutor<>(new InThreadExecutorService()), new FileDistributionFactory(new ConfigserverConfig.Builder().build()), new InMemoryFlagSource(), new InThreadExecutorService(), diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TestTenantRepository.java b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TestTenantRepository.java index 8279bf4df5e..687d58fd23b 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TestTenantRepository.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/TestTenantRepository.java @@ -50,6 +50,7 @@ public class TestTenantRepository extends TenantRepository { ConfigCurator.create(curator), metrics, new StripedExecutor<>(new InThreadExecutorService()), + new StripedExecutor<>(new InThreadExecutorService()), fileDistributionFactory, flagSource, new InThreadExecutorService(), |