aboutsummaryrefslogtreecommitdiffstats
path: root/configserver/src/main/java/com/yahoo/vespa/config
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2019-09-05 13:05:10 +0200
committerGitHub <noreply@github.com>2019-09-05 13:05:10 +0200
commitad44321fe9c09ed235754fcfc00e5f24ccc5c976 (patch)
treeff042d48497cfaa695b22c1ab84f38304a07f5ab /configserver/src/main/java/com/yahoo/vespa/config
parent1e6bd99bf983334cb3fdd7ae0e5af85461e85229 (diff)
Revert "Add config model versions to response "
Diffstat (limited to 'configserver/src/main/java/com/yahoo/vespa/config')
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java8
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/application/ApplicationSet.java4
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java11
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/RemoteSession.java3
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/tenant/TenantRequestHandler.java4
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;