Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Zookeeper, mail # dev - Re: Review Request: Dynamic reconfiguration, see https://issues.apache.org/jira/browse/ZOOKEEPER-107


+
Edward Ribeiro 2013-01-23, 07:30
+
Alexander Shraer 2013-01-25, 07:43
+
Edward Ribeiro 2013-01-23, 07:20
+
Alexander Shraer 2013-01-25, 07:43
+
Edward Ribeiro 2013-01-23, 07:35
+
Edward Ribeiro 2013-01-23, 07:32
+
Edward Ribeiro 2013-01-23, 07:08
+
Alexander Shraer 2013-01-25, 07:43
+
Edward Ribeiro 2013-01-23, 06:57
+
Alexander Shraer 2013-01-25, 07:59
+
fpj@... 2013-01-31, 00:58
Copy link to this message
-
Re: Review Request: Dynamic reconfiguration, see https://issues.apache.org/jira/browse/ZOOKEEPER-107
Alexander Shraer 2013-01-31, 17:05


> On Jan. 31, 2013, 12:58 a.m., fpj wrote:
> > /src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java, line 243
> > <https://reviews.apache.org/r/6707/diff/5/?file=251937#file251937line243>
> >
> >     I guess that the problem of having the quorum verifier info into stat is that we would have to ship it with every response? I was really looking for a way of removing the reference to QuorumZooKeeperServer.

Just in responses for getData and reconfig, where you probably want to know anyway what happened. For example in incremental reconfig you just say the delta but don't know the final state.
> On Jan. 31, 2013, 12:58 a.m., fpj wrote:
> > /src/java/main/org/apache/zookeeper/server/quorum/FastLeaderElection.java, line 267
> > <https://reviews.apache.org/r/6707/diff/5/?file=251945#file251945line267>
> >
> >     Could you explain me why you're shutting down leader election here? How can we get into this scenario?

This is when during leader election you discover a server from a future configuration. For example the new configuration were activated while you were down. In this case you may be not talking to the right set of servers. We update the set of servers and restart leader election so that you connect to the right peers. The configuration info is now sent in all FLE notification messages for this purpose.

In general, in several occasions (upon every "serious" change in configuration) I restart leader election. In many places this can be avoided, for example by connecting to new peers, or modifying existing connections. So restarting is often done for simplicity to reuse existing code.
- Alexander
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6707/#review15907
-----------------------------------------------------------
On Jan. 25, 2013, 7:59 a.m., Alexander Shraer wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6707/
> -----------------------------------------------------------
>
> (Updated Jan. 25, 2013, 7:59 a.m.)
>
>
> Review request for zookeeper.
>
>
> Description
> -------
>
> see https://issues.apache.org/jira/browse/ZOOKEEPER-107
>
>
> Diffs
> -----
>
>   /src/c/include/proto.h 1438352
>   /src/c/include/zookeeper.h 1438352
>   /src/c/src/cli.c 1438352
>   /src/c/src/zookeeper.c 1438352
>   /src/java/main/org/apache/zookeeper/KeeperException.java 1438352
>   /src/java/main/org/apache/zookeeper/ZooDefs.java 1438352
>   /src/java/main/org/apache/zookeeper/ZooKeeper.java 1438352
>   /src/java/main/org/apache/zookeeper/ZooKeeperMain.java 1438352
>   /src/java/main/org/apache/zookeeper/cli/GetConfigCommand.java PRE-CREATION
>   /src/java/main/org/apache/zookeeper/cli/ReconfigCommand.java PRE-CREATION
>   /src/java/main/org/apache/zookeeper/common/StringUtils.java 1438352
>   /src/java/main/org/apache/zookeeper/server/DataTree.java 1438352
>   /src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java 1438352
>   /src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 1438348
>   /src/java/main/org/apache/zookeeper/server/NettyServerCnxnFactory.java 1438352
>   /src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java 1438352
>   /src/java/main/org/apache/zookeeper/server/Request.java 1438352
>   /src/java/main/org/apache/zookeeper/server/ServerCnxnFactory.java 1438352
>   /src/java/main/org/apache/zookeeper/server/TraceFormatter.java 1438352
>   /src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java 1438352
>   /src/java/main/org/apache/zookeeper/server/quorum/FastLeaderElection.java 1438352
>   /src/java/main/org/apache/zookeeper/server/quorum/Follower.java 1438352
>   /src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java 1438352
>   /src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1438352
+
Alexander Shraer 2013-02-05, 05:07
+
Alexander Shraer 2013-02-06, 01:00
+
Edward Ribeiro 2013-03-22, 23:32