aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
diff options
context:
space:
mode:
authorGravatar David G. Quintas <dgq@google.com>2018-08-15 17:58:20 -0700
committerGravatar GitHub <noreply@github.com>2018-08-15 17:58:20 -0700
commita45e38622edd55f2b5c7fea0abf53f35ab68021b (patch)
tree78a630014cb9f164a2092e0423fcfd75c25585a2 /src
parent2cb3e823bf052a77aedfea11de9a375314c605bc (diff)
parentd08ea3025b653d63652b65d52fa58517bcfa94e5 (diff)
Merge pull request #16054 from dgquintas/pf_ppicks_fix
Fixed ordering in adding pending picks to PF
Diffstat (limited to 'src')
-rw-r--r--src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc b/src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc
index 2b6a9ba8c5..bc51903ef5 100644
--- a/src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc
+++ b/src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc
@@ -279,11 +279,11 @@ bool PickFirst::PickLocked(PickState* pick, grpc_error** error) {
"No pick result available but synchronous result required.");
return true;
}
+ pick->next = pending_picks_;
+ pending_picks_ = pick;
if (!started_picking_) {
StartPickingLocked();
}
- pick->next = pending_picks_;
- pending_picks_ = pick;
return false;
}