flasn4nt
/
grpc-java
mirror of https://github.com/grpc/grpc-java.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

util: Mark GracefulSwitchLB.switchTo() deprecated

This commit is contained in:
Eric Anderson 2024-07-22 13:09:42 -07:00
parent 9269050125
commit 85e0a01ecd
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
util/src/main/java/io/grpc/util/GracefulSwitchLoadBalancer.java
View File

@ -142,7 +142,11 @@ public final class GracefulSwitchLoadBalancer extends ForwardingLoadBalancer {
/**
* Gracefully switch to a new policy defined by the given factory, if the given factory isn't
* equal to the current one.
*
* @deprecated Use {@code parseLoadBalancingPolicyConfig()} and pass the configuration to
* {@link io.grpc.LoadBalancer.ResolvedAddresses.Builder#setLoadBalancingPolicyConfig}
*/
@Deprecated
public void switchTo(LoadBalancer.Factory newBalancerFactory) {
switchToCalled = true;
switchToInternal(newBalancerFactory);

13
util/src/test/java/io/grpc/util/GracefulSwitchLoadBalancerTest.java
View File

@ -86,6 +86,7 @@ public class GracefulSwitchLoadBalancerTest {
// OLD TESTS
@Test
@Deprecated
public void switchTo_canHandleEmptyAddressListFromNameResolutionForwardedToLatestPolicy() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -115,6 +116,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_handleResolvedAddressesAndNameResolutionErrorForwardedToLatestPolicy() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -155,6 +157,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_acceptResolvedAddressesAndNameResolutionErrorForwardedToLatestPolicy() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -195,6 +198,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_shutdownTriggeredWhenSwitchAndForwardedWhenSwitchLbShutdown() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -226,6 +230,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_requestConnectionForwardedToLatestPolicies() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -263,6 +268,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_createSubchannelForwarded() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -289,6 +295,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_updateBalancingStateIsGraceful() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -340,6 +347,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_switchWhileOldPolicyIsNotReady() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -368,6 +376,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_switchWhileOldPolicyGoesFromReadyToNotReady() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -398,6 +407,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_switchWhileOldPolicyGoesFromReadyToNotReadyWhileNewPolicyStillIdle() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -428,6 +438,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_newPolicyNameTheSameAsPendingPolicy_shouldHaveNoEffect() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -445,6 +456,7 @@ public class GracefulSwitchLoadBalancerTest {
}
@Test
@Deprecated
public void switchTo_newPolicyNameTheSameAsCurrentPolicy_shouldShutdownPendingLb() {
gracefulSwitchLb.switchTo(lbPolicies[0]);
LoadBalancer lb0 = balancers.get(lbPolicies[0]);
@ -468,6 +480,7 @@ public class GracefulSwitchLoadBalancerTest {
@Test
@Deprecated
public void switchTo_newLbFactoryEqualToOldOneShouldHaveNoEffect() {
final List<LoadBalancer> balancers = new ArrayList<>();