Skip to content

core: keep round_robin lb subchannel in TRANSIENT_FAILURE until becoming READY #6657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
voidzcy merged 9 commits into from
Apr 6, 2020
Merged

core: keep round_robin lb subchannel in TRANSIENT_FAILURE until becoming READY #6657

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
246c81b
Make round_robin subchannel stay in TRANSIENT_FAILURE until becoming …
voidzcy Jan 29, 2020
60f8f9d
Merge branch 'master' of github.com:grpc/grpc-java into bugfix/round_…
voidzcy Apr 2, 2020
4cf7210
Fix wrong git merge
voidzcy Apr 3, 2020
b3fd8a7
Only need to do state transition based on RR LB's state without looki…
voidzcy Apr 3, 2020
654dd7b
May still want to update picker even if state has not change to ready.
voidzcy Apr 3, 2020
1108da5
Manage each subchannel's state, as they operates individually.
voidzcy Apr 3, 2020
72e60ab
Improve test to reflect each subchannel operates individually.
voidzcy Apr 3, 2020
348614d
Fix existing test for verifying subchannel state after TF.
voidzcy Apr 3, 2020
5c91cbd
Make verification for subchannel state more explicit.
voidzcy Apr 3, 2020
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions core/src/main/java/io/grpc/util/RoundRobinLoadBalancer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -197,6 +197,9 @@ private void updateBalancingState() {

private void updateBalancingState(ConnectivityState state, RoundRobinPicker picker) {
if (state != currentState || !picker.isEquivalentTo(currentPicker)) {
if (currentState == TRANSIENT_FAILURE && state != READY) {
return;
}
helper.updateBalancingState(state, picker);
currentState = state;
currentPicker = picker;
Expand Down
39 changes: 39 additions & 0 deletions core/src/test/java/io/grpc/util/RoundRobinLoadBalancerTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -294,6 +294,45 @@ public void pickAfterStateChange() throws Exception {
verifyNoMoreInteractions(mockHelper);
}

@Test
public void stayTransientFailureUntilReady() {
InOrder inOrder = inOrder(mockHelper);
loadBalancer.handleResolvedAddresses(
ResolvedAddresses.newBuilder().setAddresses(servers).setAttributes(Attributes.EMPTY)
.build());

inOrder.verify(mockHelper).updateBalancingState(eq(CONNECTING), isA(EmptyPicker.class));

for (Subchannel sc : loadBalancer.getSubchannels()) {
deliverSubchannelState(
sc,
ConnectivityStateInfo
.forTransientFailure(Status.UNKNOWN.withDescription("connection broken")));
}
inOrder.verify(mockHelper).updateBalancingState(eq(TRANSIENT_FAILURE), isA(EmptyPicker.class));

for (Subchannel sc : loadBalancer.getSubchannels()) {
deliverSubchannelState(
sc,
ConnectivityStateInfo.forNonError(IDLE));
}
inOrder.verifyNoMoreInteractions();

for (Subchannel sc : loadBalancer.getSubchannels()) {
deliverSubchannelState(
sc,
ConnectivityStateInfo.forNonError(CONNECTING));
}
inOrder.verifyNoMoreInteractions();

Subchannel subchannel = loadBalancer.getSubchannels().iterator().next();
deliverSubchannelState(subchannel, ConnectivityStateInfo.forNonError(READY));
inOrder.verify(mockHelper).updateBalancingState(eq(READY), isA(ReadyPicker.class));

verify(mockHelper, times(3)).createSubchannel(any(CreateSubchannelArgs.class));
verifyNoMoreInteractions(mockHelper);
}

@Test
public void pickerRoundRobin() throws Exception {
Subchannel subchannel = mock(Subchannel.class);
Expand Down