diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-09-05 13:05:10 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-05 13:05:10 +0200 |
commit | ad44321fe9c09ed235754fcfc00e5f24ccc5c976 (patch) | |
tree | ff042d48497cfaa695b22c1ab84f38304a07f5ab /configserver/src/main/java/com/yahoo/vespa/config | |
parent | 1e6bd99bf983334cb3fdd7ae0e5af85461e85229 (diff) |
Revert "Add config model versions to response "
Diffstat (limited to 'configserver/src/main/java/com/yahoo/vespa/config')
5 files changed, 7 insertions, 23 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java index 3c95e008f7a..0e17a31886b 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java @@ -395,14 +395,6 @@ public class ApplicationRepository implements com.yahoo.config.provision.Deploye return getApplication(applicationId).getApplicationGeneration(); } - public List<Version> getApplicationVersions(ApplicationId applicationId) { - Tenant tenant = tenantRepository.getTenant(applicationId.tenant()); - if (tenant == null) throw new NotFoundException("Tenant '" + applicationId.tenant() + "' not found"); - long sessionId = getSessionIdForApplication(tenant, applicationId); - RemoteSession session = tenant.getRemoteSessionRepo().getSession(sessionId); - return session.applicationSet().getAllApplicationVersions(); - } - public void restart(ApplicationId applicationId, HostFilter hostFilter) { hostProvisioner.ifPresent(provisioner -> provisioner.restart(applicationId, hostFilter)); } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationSet.java b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationSet.java index 7c429ed9dcd..41119077b28 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationSet.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationSet.java @@ -96,8 +96,4 @@ public final class ApplicationSet { return new ArrayList<>(applications.values()); } - public List<Version> getAllApplicationVersions() { - return new ArrayList<>(applications.keySet()); - } - } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java index c9657175eaf..e18c6ad6c56 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java @@ -13,7 +13,6 @@ import com.yahoo.container.jdisc.HttpRequest; import com.yahoo.container.jdisc.HttpResponse; import com.yahoo.jdisc.Response; import com.yahoo.jdisc.application.BindingMatch; -import com.yahoo.slime.Cursor; import com.yahoo.vespa.config.server.ApplicationRepository; import com.yahoo.vespa.config.server.http.ContentHandler; import com.yahoo.vespa.config.server.http.ContentRequest; @@ -21,9 +20,9 @@ import com.yahoo.vespa.config.server.http.HttpErrorResponse; import com.yahoo.vespa.config.server.http.HttpHandler; import com.yahoo.vespa.config.server.http.JSONResponse; import com.yahoo.vespa.config.server.http.NotFoundException; +import com.yahoo.vespa.config.server.tenant.Tenant; import java.time.Duration; -import java.util.List; import java.util.Optional; /** @@ -113,9 +112,7 @@ public class ApplicationHandler extends HttpHandler { return new ApplicationSuspendedResponse(applicationRepository.isSuspended(applicationId)); } - return new GetApplicationResponse(Response.Status.OK, - applicationRepository.getApplicationGeneration(applicationId), - applicationRepository.getApplicationVersions(applicationId)); + return new GetApplicationResponse(Response.Status.OK, applicationRepository.getApplicationGeneration(applicationId)); } @Override @@ -247,11 +244,9 @@ public class ApplicationHandler extends HttpHandler { } private static class GetApplicationResponse extends JSONResponse { - GetApplicationResponse(int status, long generation, List<Version> versions) { + GetApplicationResponse(int status, long generation) { super(status); object.setLong("generation", generation); - Cursor versionsArray = object.setArray("configModelVersions"); - versions.forEach(version -> versionsArray.addString(version.toFullString())); } } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSession.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSession.java index b8320ea865e..26f437920ad 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSession.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSession.java @@ -134,7 +134,4 @@ public class RemoteSession extends Session { transaction.close(); } - public ApplicationSet applicationSet() { - return applicationSet; - } } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/tenant/TenantRequestHandler.java b/configserver/src/main/java/com/yahoo/vespa/config/server/tenant/TenantRequestHandler.java index 5926f64cf87..2f9a5eb9277 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/tenant/TenantRequestHandler.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/tenant/TenantRequestHandler.java @@ -2,6 +2,7 @@ package com.yahoo.vespa.config.server.tenant; import com.yahoo.component.Version; +import com.yahoo.concurrent.StripedExecutor; import com.yahoo.config.FileReference; import com.yahoo.config.provision.ApplicationId; import com.yahoo.config.provision.TenantName; @@ -19,11 +20,13 @@ import com.yahoo.vespa.config.server.application.ApplicationMapper; import com.yahoo.vespa.config.server.application.ApplicationSet; import com.yahoo.vespa.config.server.application.TenantApplications; import com.yahoo.vespa.config.server.application.VersionDoesNotExistException; +import com.yahoo.vespa.config.server.host.HostRegistries; import com.yahoo.vespa.config.server.host.HostRegistry; import com.yahoo.vespa.config.server.host.HostValidator; import com.yahoo.vespa.config.server.monitoring.MetricUpdater; import com.yahoo.vespa.config.server.monitoring.Metrics; import com.yahoo.vespa.config.server.rpc.ConfigResponseFactory; +import com.yahoo.vespa.curator.Curator; import com.yahoo.vespa.curator.Lock; import java.time.Clock; @@ -32,6 +35,7 @@ import java.util.LinkedHashSet; import java.util.List; import java.util.Optional; import java.util.Set; +import java.util.concurrent.ExecutorService; import static java.util.stream.Collectors.toSet; |