diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-03-12 11:41:31 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-03-12 11:41:31 +0100 |
commit | 32c8299692736b66c31a93f3f18d1921b559a298 (patch) | |
tree | 0f0c32302013b99c4f864a1cf6e6aa23fae06dc0 /configserver | |
parent | 69a207f3603c64a1402fb8478b49d61afb8dae55 (diff) |
Revert "Set generation in empty sentinel config response"
Diffstat (limited to 'configserver')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java | 2 | ||||
-rw-r--r-- | configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcServerTest.java | 2 |
2 files changed, 1 insertions, 3 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java index bebd0b341db..bbdef71129a 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java @@ -167,7 +167,7 @@ class GetConfigProcessor implements Runnable { log.log(LogLevel.INFO, "Returning empty sentinel config for request from " + request.getClientHostName()); ConfigPayload emptyPayload = ConfigPayload.empty(); String configMd5 = ConfigUtils.getMd5(emptyPayload); - ConfigResponse config = SlimeConfigResponse.fromConfigPayload(emptyPayload, null, request.getRequestGeneration() + 1, false, configMd5); + ConfigResponse config = SlimeConfigResponse.fromConfigPayload(emptyPayload, null, 0, false, configMd5); request.addOkResponse(request.payloadFromResponse(config), config.getGeneration(), false, config.getConfigMd5()); respond(request); } diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcServerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcServerTest.java index 959b284220b..c8bc9364922 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcServerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcServerTest.java @@ -96,8 +96,6 @@ public class RpcServerTest { new ConfigPayloadApplier<>(builder).applyPayload(payload); SentinelConfig config = new SentinelConfig(builder); assertEquals(0, config.service().size()); - // Test generation explicitly since it is not set set the usual way - assertTrue(clientReq.getNewGeneration() > clientReq.getRequestGeneration()); } } |