zookeeper-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] lvfangmin commented on issue #1047: [ZOOKEEPER-3500] Improving the ZAB UPTODATE semantic to only issue it to learner when there is limited lagging
Date Wed, 11 Sep 2019 01:08:24 GMT
lvfangmin commented on issue #1047: [ZOOKEEPER-3500] Improving the ZAB UPTODATE semantic to
only issue it to learner when there is limited lagging
URL: https://github.com/apache/zookeeper/pull/1047#issuecomment-530177202
 
 
   @eolivelli it seems tricky to test 3.5 compatibility with 3.6.
   
   @hanm here are the answers to your questions:
   
   > Are these txns queued in queuedPackets of LearnerHandler? I was thinking you referred
to the packetsNotCommitted of Learner.
   
   Yes, those txns are queued up in the LearnerHandler, since learner was busy taking snapshot
and doesn't read from socket, which put pressure back to leader and packets queued up in LearnerHandler.
   
   > One case I was thinking is when a large subset of servers lagging and recovering,
the zookeeper clients might have to wait longer to find a server to establish a connection,
so client side's retry / timeout might have to be adjusted to deal with this change.
   
   It may need to retry a few servers before find a up and running one, but it should be fine,
since less than minority servers will be in this state, and they will disconnect clients if
they haven't finished syncing with leader yet.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message