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: Add zk.updateServerList(newServerList)


+
Marshall McMullen 2012-10-16, 15:11
+
Marshall McMullen 2012-11-06, 16:05
+
Marshall McMullen 2012-11-06, 16:14
Copy link to this message
-
Re: Review Request: Add zk.updateServerList(newServerList)
Alexander Shraer 2012-11-06, 23:40

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3781/
-----------------------------------------------------------

(Updated Nov. 6, 2012, 11:40 p.m.)
Review request for zookeeper.
Changes
-------

New patch includes Marshall's changes to the C client, addressing Michi's comments
Description
-------

https://issues.apache.org/jira/browse/ZOOKEEPER-1355
Diffs (updated)
-----

  /src/c/Makefile.am 1406178
  /src/c/include/zookeeper.h 1406178
  /src/c/src/addrvec.h PRE-CREATION
  /src/c/src/addrvec.c PRE-CREATION
  /src/c/src/mt_adaptor.c 1406178
  /src/c/src/st_adaptor.c 1406178
  /src/c/src/zk_adaptor.h 1406178
  /src/c/src/zookeeper.c 1406178
  /src/c/tests/TestReconfig.cc PRE-CREATION
  /src/c/tests/TestZookeeperClose.cc 1406178
  /src/c/tests/TestZookeeperInit.cc 1406178
  /src/c/tests/ZKMocks.cc 1406178
  /src/docs/src/documentation/content/xdocs/zookeeperProgrammers.xml 1406178
  /src/java/main/org/apache/zookeeper/ZooKeeper.java 1406178
  /src/java/main/org/apache/zookeeper/client/HostProvider.java 1406178
  /src/java/main/org/apache/zookeeper/client/StaticHostProvider.java 1406178
  /src/java/test/org/apache/zookeeper/test/StaticHostProviderTest.java 1406178

Diff: https://reviews.apache.org/r/3781/diff/
Testing
-------

new tests included as part of the patch
Thanks,

Alexander Shraer

+
fpj@... 2012-11-13, 17:02
+
Marshall McMullen 2012-11-14, 06:00
+
Alexander Shraer 2012-11-14, 06:08
+
Alexander Shraer 2012-11-14, 06:54