diff options
author | Martin Polden <mpolden@mpolden.no> | 2019-04-10 09:18:45 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2019-04-10 09:34:02 +0200 |
commit | 4fac9aefba8b590a5b190ae42cdd7ae9a3fc38e9 (patch) | |
tree | c6ceadb04ab638da0717590f6492399c3121182c /controller-server/src | |
parent | 638fa12463b7f19ff1f4e1eb8301a771198b8870 (diff) |
Update doc
Diffstat (limited to 'controller-server/src')
2 files changed, 12 insertions, 13 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Endpoint.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Endpoint.java index a872421b5ea..4a0f3b34a2b 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Endpoint.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Endpoint.java @@ -59,7 +59,10 @@ public class Endpoint { return legacy; } - /** Returns whether this endpoint supports direct routing */ + /** + * Returns whether this endpoint supports direct routing. Direct routing means that this endpoint is served by an + * exclusive load balancer instead of a shared routing layer. + */ public boolean directRouting() { return directRouting; } @@ -212,12 +215,11 @@ public class Endpoint { private boolean legacy = false; private boolean directRouting = false; - private EndpointBuilder(ApplicationId application) { this.application = application; } - /** Sets the cluster and zone target of this endpoint */ + /** Sets the cluster and zone target of this */ public EndpointBuilder target(ClusterSpec.Id cluster, ZoneId zone) { if (rotation != null) { throw new IllegalArgumentException("Cannot set both cluster and rotation target"); @@ -227,7 +229,7 @@ public class Endpoint { return this; } - /** Sets the rotation target of this endpoint */ + /** Sets the rotation target of this */ public EndpointBuilder target(RotationName rotation) { if (cluster != null && zone != null) { throw new IllegalArgumentException("Cannot set both cluster and rotation target"); @@ -236,28 +238,25 @@ public class Endpoint { return this; } - /** Sets the port of this endpoint */ + /** Sets the port of this */ public EndpointBuilder on(Port port) { this.port = port; return this; } - /** Sets this endpoint as a legacy endpoint */ + /** Marks this as a legacy endpoint */ public EndpointBuilder legacy() { this.legacy = true; return this; } - /** - * Sets direct routing support for this endpoint. Direct routing means that this endpoint is served by an - * exclusive load balancer instead of a shared routing layer. - */ + /** Enables direct routing support for this */ public EndpointBuilder directRouting() { this.directRouting = true; return this; } - /** The system where this endpoint is available */ + /** Sets the system that owns this */ public Endpoint in(SystemName system) { String name; if (cluster != null && zone != null) { diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/RoutingPolicy.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/RoutingPolicy.java index 9cf4e4a9254..c4b69ce5588 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/RoutingPolicy.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/RoutingPolicy.java @@ -20,7 +20,7 @@ import java.util.Set; * @author mortent * @author mpolden */ -public class RoutingPolicy { +public class RoutingPolicy { private final ApplicationId owner; private final ZoneId zone; @@ -107,7 +107,7 @@ public class RoutingPolicy { return Endpoint.of(application).target(rotation).on(Port.tls()).directRouting().in(system); } - /** Returns the endpoint of given clsuter */ + /** Returns the endpoint of given cluster */ public static Endpoint endpointOf(ClusterSpec.Id cluster, ApplicationId application, ZoneId zone, SystemName system) { return Endpoint.of(application).target(cluster, zone).on(Port.tls()).directRouting().in(system); } |