diff options
author | Harald Musum <musum@verizonmedia.com> | 2019-08-28 08:04:04 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-08-28 08:04:04 +0200 |
commit | f123f4095ceb696159c4753c25488874b3cb8fe3 (patch) | |
tree | 5bef15c6ec83afc56828c160b4042f0a18cc05f4 | |
parent | bd65431ca39696ec242d63e4d378bae1a333e0c9 (diff) |
Revert "Revert " Upgrade to zookeeper 3.4.14, take 4""
-rw-r--r-- | application/pom.xml | 5 | ||||
-rw-r--r-- | athenz-identity-provider-service/pom.xml | 7 | ||||
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/serviceview/Cluster.java | 8 | ||||
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/serviceview/Service.java | 21 | ||||
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/serviceview/ServiceModel.java | 15 | ||||
-rw-r--r-- | node-admin/pom.xml | 7 | ||||
-rw-r--r-- | parent/pom.xml | 4 | ||||
-rw-r--r-- | serviceview/src/main/java/com/yahoo/vespa/serviceview/bindings/ServicePort.java | 6 | ||||
-rw-r--r-- | zkfacade/pom.xml | 4 |
9 files changed, 37 insertions, 40 deletions
diff --git a/application/pom.xml b/application/pom.xml index 20b754fc236..5a370e9ffe6 100644 --- a/application/pom.xml +++ b/application/pom.xml @@ -54,11 +54,6 @@ <artifactId>container-search-gui</artifactId> <version>${project.version}</version> </dependency> - <dependency> - <groupId>com.yahoo.vespa</groupId> - <artifactId>zkfacade</artifactId> - <version>${project.version}</version> - </dependency> <!-- Because these are provided in jdisc_http_service and just preinstalled: --> <dependency> <groupId>com.yahoo.vespa</groupId> diff --git a/athenz-identity-provider-service/pom.xml b/athenz-identity-provider-service/pom.xml index 1b1edcc64b8..501904a1d62 100644 --- a/athenz-identity-provider-service/pom.xml +++ b/athenz-identity-provider-service/pom.xml @@ -127,6 +127,13 @@ <artifactId>mockito-core</artifactId> <scope>test</scope> </dependency> + <dependency> + <!-- Needed for node repo mock --> + <groupId>com.yahoo.vespa</groupId> + <artifactId>zkfacade</artifactId> + <version>${project.version}</version> + <scope>test</scope> + </dependency> </dependencies> <build> diff --git a/configserver/src/main/java/com/yahoo/vespa/serviceview/Cluster.java b/configserver/src/main/java/com/yahoo/vespa/serviceview/Cluster.java index 3b39a85c6b1..d7005de5b07 100644 --- a/configserver/src/main/java/com/yahoo/vespa/serviceview/Cluster.java +++ b/configserver/src/main/java/com/yahoo/vespa/serviceview/Cluster.java @@ -6,22 +6,18 @@ import java.util.List; import com.google.common.collect.ImmutableList; -import edu.umd.cs.findbugs.annotations.NonNull; - /** * Model a single cluster of services in the Vespa model. * - * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a> + * @author Steinar Knutsen */ public final class Cluster implements Comparable<Cluster> { - @NonNull + public final String name; - @NonNull public final String type; /** * An ordered list of the service instances in this cluster. */ - @NonNull public final ImmutableList<Service> services; public Cluster(String name, String type, List<Service> services) { diff --git a/configserver/src/main/java/com/yahoo/vespa/serviceview/Service.java b/configserver/src/main/java/com/yahoo/vespa/serviceview/Service.java index b5d18bdbb1f..f10e2afa046 100644 --- a/configserver/src/main/java/com/yahoo/vespa/serviceview/Service.java +++ b/configserver/src/main/java/com/yahoo/vespa/serviceview/Service.java @@ -1,37 +1,31 @@ // Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.serviceview; +import com.google.common.collect.ImmutableList; +import com.yahoo.text.Utf8; + import java.math.BigInteger; import java.security.MessageDigest; import java.security.NoSuchAlgorithmException; import java.util.ArrayList; import java.util.List; -import com.google.common.collect.ImmutableList; -import com.yahoo.text.Utf8; - -import edu.umd.cs.findbugs.annotations.NonNull; - /** * Model a single service instance as a sortable object. * - * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a> + * @author Steinar Knutsen */ public final class Service implements Comparable<Service> { - @NonNull + public final String serviceType; - @NonNull public final String host; public final int statePort; - @NonNull public final String configId; - @NonNull public final List<Integer> ports; - @NonNull public final String name; public Service(String serviceType, String host, int statePort, String clusterName, String clusterType, - String configId, List<Integer> ports, String name) { + String configId, List<Integer> ports, String name) { this.serviceType = serviceType; this.host = host.toLowerCase(); this.statePort = statePort; @@ -59,8 +53,7 @@ public final class Service implements Comparable<Service> { * Generate an identifier string for one of the ports of this service * suitable for using in an URL. * - * @param port - * port which this identifier pertains to + * @param port port which this identifier pertains to * @return an opaque identifier string for this service */ public String getIdentifier(int port) { diff --git a/configserver/src/main/java/com/yahoo/vespa/serviceview/ServiceModel.java b/configserver/src/main/java/com/yahoo/vespa/serviceview/ServiceModel.java index c9574ffaa7e..dc6fbb7ebd3 100644 --- a/configserver/src/main/java/com/yahoo/vespa/serviceview/ServiceModel.java +++ b/configserver/src/main/java/com/yahoo/vespa/serviceview/ServiceModel.java @@ -1,12 +1,6 @@ // Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.serviceview; -import java.util.ArrayList; -import java.util.Collections; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - import com.google.common.collect.HashBasedTable; import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableMap; @@ -19,7 +13,11 @@ import com.yahoo.vespa.serviceview.bindings.ModelResponse; import com.yahoo.vespa.serviceview.bindings.ServicePort; import com.yahoo.vespa.serviceview.bindings.ServiceView; -import edu.umd.cs.findbugs.annotations.NonNull; +import java.util.ArrayList; +import java.util.Collections; +import java.util.HashMap; +import java.util.List; +import java.util.Map; import static com.yahoo.config.model.api.container.ContainerServiceType.CLUSTERCONTROLLER_CONTAINER; @@ -37,7 +35,6 @@ public final class ServiceModel { /** * An ordered list of the clusters in this config model. */ - @NonNull public final ImmutableList<Cluster> clusters; ServiceModel(ModelResponse modelConfig) { @@ -155,7 +152,7 @@ public final class ServiceModel { } } - private Service getFirstServiceInstanceByType(@NonNull String typeName) { + private Service getFirstServiceInstanceByType(String typeName) { for (Cluster c : clusters) { for (Service s : c.services) { if (typeName.equals(s.serviceType)) { diff --git a/node-admin/pom.xml b/node-admin/pom.xml index 1188cd9fb76..7cc4c22b417 100644 --- a/node-admin/pom.xml +++ b/node-admin/pom.xml @@ -137,6 +137,13 @@ <version>${project.version}</version> <scope>test</scope> </dependency> + <!-- Needed for node repo mock --> + <dependency> + <groupId>com.yahoo.vespa</groupId> + <artifactId>zkfacade</artifactId> + <version>${project.version}</version> + <scope>test</scope> + </dependency> </dependencies> <build> <plugins> diff --git a/parent/pom.xml b/parent/pom.xml index a81f64d1725..c89d53e5160 100644 --- a/parent/pom.xml +++ b/parent/pom.xml @@ -495,14 +495,14 @@ <artifactId>commons-pool</artifactId> <version>1.5.6</version> </dependency> - <!-- Explicitly included to get Zookeeper version 3.4.13, + <!-- Explicitly included to get Zookeeper version 3.4.14, can be excluded if you want the Zookeeper version used by curator by default --> <dependency> <groupId>org.apache.zookeeper</groupId> <artifactId>zookeeper</artifactId> - <version>3.4.13</version> + <version>3.4.14</version> </dependency> <dependency> <groupId>org.apache.curator</groupId> diff --git a/serviceview/src/main/java/com/yahoo/vespa/serviceview/bindings/ServicePort.java b/serviceview/src/main/java/com/yahoo/vespa/serviceview/bindings/ServicePort.java index a82069ea01d..98db96a902a 100644 --- a/serviceview/src/main/java/com/yahoo/vespa/serviceview/bindings/ServicePort.java +++ b/serviceview/src/main/java/com/yahoo/vespa/serviceview/bindings/ServicePort.java @@ -6,13 +6,11 @@ import java.util.List; import com.fasterxml.jackson.annotation.JsonIgnoreProperties; import com.google.common.base.Splitter; -import edu.umd.cs.findbugs.annotations.NonNull; - /** * View of {@link com.yahoo.cloud.config.ModelConfig.Hosts.Services.Ports}. * * @author mortent - * @author <a href="mailto:steinar@yahoo-inc.com">Steinar Knutsen</a> + * @author Steinar Knutsen */ @JsonIgnoreProperties(value = { "splitOnSpace" }, ignoreUnknown = true) public class ServicePort { @@ -28,7 +26,7 @@ public class ServicePort { * @return true if all argument tags are present for this port, false * otherwise */ - public boolean hasTags(@NonNull String... tag) { + public boolean hasTags(String... tag) { if (tags == null) { return false; } diff --git a/zkfacade/pom.xml b/zkfacade/pom.xml index c57ab45e6c6..2e016e71132 100644 --- a/zkfacade/pom.xml +++ b/zkfacade/pom.xml @@ -65,6 +65,10 @@ <version>${project.version}</version> <scope>provided</scope> </dependency> + <dependency> + <groupId>org.apache.zookeeper</groupId> + <artifactId>zookeeper</artifactId> + </dependency> </dependencies> <build> <plugins> |