Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886375
In src/java/main/org/apache/zookeeper/ZooKeeper.java:
In src/java/main/org/apache/zookeeper/ZooKeeper.java on line 1538:
Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886369
In src/java/main/org/apache/zookeeper/ZooKeeper.java:
In src/java/main/org/apache/zookeeper/ZooKeeper.java on line 1439:
Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886377
In src/java/main/org/apache/zookeeper/ZooKeeper.java:
In src/java/main/org/apache/zookeeper/ZooKeeper.java on line 2380:
Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886435
In src/java/test/org/apache/zookeeper/test/ACLTest.java:
In src/java/test/org/apache/zookeeper/test/ACLTest.java on line 21
Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886495
In src/java/test/org/apache/zookeeper/test/ACLTest.java:
In src/java/test/org/apache/zookeeper/test/ACLTest.java on line 19
Github user eribeiro commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22886506
In src/java/test/org/apache/zookeeper/test/ACLTest.java:
In src/java/test/org/apache/zookeeper/test/ACLTest.java on line 23
Github user jainbhupendra24 commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22943164
In src/java/test/org/apache/zookeeper/test/ACLTest.java:
In src/java/test/org/apache/zookeeper/test/ACLTest.java on
Github user jainbhupendra24 commented on the pull request:
https://github.com/apache/zookeeper/commit/15d9a574edeaeef1c43937833cef44584115cd3d#commitcomment-22946367
In src/java/test/org/apache/zookeeper/test/ACLTest.java:
In src/java/test/org/apache/zookeeper/test/ACLTest.java on
Github user meteorfreeze1234 commented on the pull request:
https://github.com/apache/zookeeper/commit/3952f4579cb55602fecc002804e3d40aa1196ccc#commitcomment-29479708
dfsfdsf
---
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/cac68c3503d207c5f7d955ba582dfe3bfa22cb9b#commitcomment-29499666
Interesting so the test used 3 zeroes
---
Github user nkalmar commented on the pull request:
https://github.com/apache/zookeeper/commit/cac68c3503d207c5f7d955ba582dfe3bfa22cb9b#commitcomment-29499851
Yes, but the intention was to use unique port numbers, although the test
still passed with 3 zeros.
Also, I'm not sure why
Github user maoling commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29873077
@nkalmar Could you plz create a JIRA to document this metric
`zk_fsync_threshold_exceed_count` in the part of `mntr` of
`zoo
Github user nkalmar commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29880817
@maoling It has been intentionally removed from 3.6 and 3.5, as I was told
the 4 letter words is deprecated, and it will be rem
Github user maoling commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29885683
@nkalmar
Thanks for your ACK. 4 letter words will be deprecated? why?
---
Github user anmolnar commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29886282
Insecure. JMX and Jetty are the future for 3.5+
JMX is also questionable. Not sure if we could drop that too.
---
Github user maoling commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29915646
@anmolnar
Sir, Could you plz tell me why the 4 letter words is insecure?
---
Github user nkalmar commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29917129
@maoling , the problem is, there is no security implemented. Anyone user
who can access ZooKeeper, can send commands to the ens
Github user maoling commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29922823
@nkalmar Thanks for your nice explain.
these security problems can also exist in the JMX and Jetty?
---
Github user nkalmar commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29934308
Both JMX and Jetty can be secured. The problem here is, as of my
understanding, is that 4ltw command uses the client port. You
Github user maoling commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29935400
wonderful explain.thanks @nkalmar
---
Github user anmolnar commented on the pull request:
https://github.com/apache/zookeeper/commit/a2623a625a4778720f7d5482d0a66e9b37ae556f#commitcomment-29970536
@maoling Trying to find the best email thread in MarkMail for you, but
@nkalmar gave you a very good reasoning. It's a long ou
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/c57b62d57b8b22d568b1cea3b8039566936e0558#commitcomment-30735811
In src/java/main/org/apache/zookeeper/SaslServerPrincipal.java:
In src/java/main/org/apache/zookeeper/SaslServerPrincipal.
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31139343
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31139412
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31139429
In
zookeeper-server/src/test/java/org/apache/zookeeper/common/FileChangeWatcherTest.java:
In
zookeeper-server/src/test/j
Github user ivmaykov commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31143262
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/z
Github user ivmaykov commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31143267
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/z
Github user ivmaykov commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31143275
In
zookeeper-server/src/test/java/org/apache/zookeeper/common/FileChangeWatcherTest.java:
In
zookeeper-server/src/test/ja
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31151383
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31151424
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31151441
In
zookeeper-server/src/test/java/org/apache/zookeeper/common/FileChangeWatcherTest.java:
In
zookeeper-server/src/test/j
Github user ivmaykov commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31151782
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/z
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/232232e7f338a9eca440d89dcfc8ed22e0336e60#commitcomment-31151817
In zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java:
In zookeeper-server/src/main/java/org/apache/
Github user eolivelli commented on the pull request:
https://github.com/apache/zookeeper/commit/82da83a7176bbdd73dedb6fc3d96dac706cefad3#commitcomment-31339939
In zookeeper-server/src/main/java/org/apache/zookeeper/jmx/ManagedUtil.java:
In zookeeper-server/src/main/java/org/apache/
Github user asdf2014 commented on the pull request:
https://github.com/apache/zookeeper/commit/82da83a7176bbdd73dedb6fc3d96dac706cefad3#commitcomment-31341678
In zookeeper-server/src/main/java/org/apache/zookeeper/jmx/ManagedUtil.java:
In zookeeper-server/src/main/java/org/apache/z
Github user sgrillon14 commented on the pull request:
https://github.com/apache/zookeeper/commit/eb38f5b5db860a35c58dde976dce540d7958fdba#commitcomment-20356354
Hello cnauroth,
You change "Win32 is supported as a **development platform only** for both
server and client." by "
Github user cnauroth commented on the pull request:
https://github.com/apache/zookeeper/commit/eb38f5b5db860a35c58dde976dce540d7958fdba#commitcomment-20365123
Hello @sgrillon14 . I am aware of multiple projects running ZooKeeper on
Windows in production. (Unfotunately, I can't reall
Github user shralex commented on the pull request:
https://github.com/apache/zookeeper/commit/de367bf0694ba8adb11d42b4fc07f6e6a851b782#commitcomment-20584642
In src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java:
In src/java/main/org/apache/zookeeper/server/quor
Github user hanm commented on the pull request:
https://github.com/apache/zookeeper/commit/de367bf0694ba8adb11d42b4fc07f6e6a851b782#commitcomment-20584729
In src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java:
In src/java/main/org/apache/zookeeper/server/quorum/
Github user hanm commented on the pull request:
https://github.com/apache/zookeeper/commit/de367bf0694ba8adb11d42b4fc07f6e6a851b782#commitcomment-20585960
In src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java:
In src/java/main/org/apache/zookeeper/server/quorum/
GitHub user Felix-Pobaby opened a pull request:
https://github.com/apache/zookeeper/pull/69
Master
Can'I update the ZooKeeper Sessions state diagram?
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Felix-Pobaby/zookeeper master
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/269
Sugon
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JiangJiafu/zookeeper sugon
Alternatively you can review and apply these changes as
Github user JiangJiafu closed the pull request at:
https://github.com/apache/zookeeper/pull/269
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user brettKK closed the pull request at:
https://github.com/apache/zookeeper/pull/494
---
GitHub user roodkcab opened a pull request:
https://github.com/apache/zookeeper/pull/646
Reconnect
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/roodkcab/zookeeper reconnect
Alternatively you can review and apply these changes
Github user roodkcab closed the pull request at:
https://github.com/apache/zookeeper/pull/646
---
GitHub user stanlyDoge opened a pull request:
https://github.com/apache/zookeeper/pull/732
typo
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stanlyDoge/zookeeper patch-1
Alternatively you can review and apply these changes as
GitHub user leoricxu opened a pull request:
https://github.com/apache/zookeeper/pull/35
Branch 3.5
git-svn-id:
https://svn.apache.org/repos/asf/zookeeper/branches/branch-3.5@1684112
13f79535-47bb-0310-9956-ffa450edef68
You can merge this pull request into a Git repository by runni
GitHub user YvonneIronberg opened a pull request:
https://github.com/apache/zookeeper/pull/41
ZOOKEEPER-2255
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/YvonneIronberg/zookeeper zk-2255
Alternatively you can review and apply
GitHub user stevejs opened a pull request:
https://github.com/apache/zookeeper/pull/47
Update zookeeperOver.html
Type fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stevejs/zookeeper patch-1
Alternatively you can review and
GitHub user rakeshadr opened a pull request:
https://github.com/apache/zookeeper/pull/49
Zookeeper 1045
QuorumPeer mutual auth using SASL - Kerberos, DIgest mechanisms.
TODO: Upgrade path has to be discussed.
You can merge this pull request into a Git repository by running:
GitHub user itayB opened a pull request:
https://github.com/apache/zookeeper/pull/56
Update zookeeper.h
Typo fix (Comments)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/itayB/zookeeper trunk
Alternatively you can review and ap
GitHub user karanmarjara opened a pull request:
https://github.com/apache/zookeeper/pull/60
Branch 3.1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/zookeeper branch-3.1
Alternatively you can review and apply these chan
GitHub user rakeshadr opened a pull request:
https://github.com/apache/zookeeper/pull/65
ZOOKEEPER-2247
Created this PR using proposed "ZOOKEEPER-2247-15.patch" in the jira.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rakeshad
Github user ralic closed the pull request at:
https://github.com/apache/zookeeper/pull/66
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
GitHub user ralic opened a pull request:
https://github.com/apache/zookeeper/pull/66
Branch 3.4
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/zookeeper branch-3.4
Alternatively you can review and apply these changes as
GitHub user buptUnixGuys opened a pull request:
https://github.com/apache/zookeeper/pull/67
Update Op.java
A minor bug.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/buptUnixGuys/zookeeper patch-1
Alternatively you can review a
GitHub user kouchengkun opened a pull request:
https://github.com/apache/zookeeper/pull/12
Update NIOServerCnxnFactory.java
workPool may be multi-thread, so the IO event of different connection
should be handled by different thread.
You can merge this pull request into a Git reposi
GitHub user tianrobin opened a pull request:
https://github.com/apache/zookeeper/pull/226
merge Request
merge from
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tianrobin/zookeeper trunk
Alternatively you can review and apply
Github user tianrobin closed the pull request at:
https://github.com/apache/zookeeper/pull/226
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature i
Github user jeffwidman closed the pull request at:
https://github.com/apache/zookeeper/pull/210
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user gstein closed the pull request at:
https://github.com/apache/zookeeper/pull/206
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is e
Github user gstein closed the pull request at:
https://github.com/apache/zookeeper/pull/212
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is e
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/250
Branch 3.4
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/zookeeper branch-3.4
Alternatively you can review and apply these chang
Github user JiangJiafu closed the pull request at:
https://github.com/apache/zookeeper/pull/250
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/253
ZOOKEEPER-2774
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JiangJiafu/zookeeper ZOOKEEPER-2774
Alternatively you can review and apply
Github user afine commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/253#discussion_r116588671
--- Diff: src/java/main/org/apache/zookeeper/ZKUtil.java ---
@@ -120,5 +120,4 @@ public static void deleteRecursive(ZooKeeper zk, final
String pathRoot, Voi
Github user JiangJiafu closed the pull request at:
https://github.com/apache/zookeeper/pull/253
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/261
Branch 3.4
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JiangJiafu/zookeeper branch-3.4
Alternatively you can review and apply these c
Github user JiangJiafu closed the pull request at:
https://github.com/apache/zookeeper/pull/261
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user maoling opened a pull request:
https://github.com/apache/zookeeper/pull/308
ZOOKEEPER-2842
1.optimize the finish() of Send/RecvWorker in QuorumCnxManager
2.remove testInitiateConnection()
3. formates some codes
You can merge this pull request into a Git reposit
GitHub user 20100507 opened a pull request:
https://github.com/apache/zookeeper/pull/376
Update OutputArchive.java
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/20100507/zookeeper master
Alternatively you can review and apply
GitHub user Shonminh opened a pull request:
https://github.com/apache/zookeeper/pull/380
Develeper shonminh
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/BeautifulCodingLife/zookeeper
develeper-shonminh
Alternatively you can
GitHub user proller opened a pull request:
https://github.com/apache/zookeeper/pull/386
Cmake fixes
1. decrease cmake version (to allow compile on some old systems)
2. do not use global include_directories (and auto include when linking to
external projects)
3. replace CMAKE
Github user phunt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/386#discussion_r141674581
--- Diff: src/c/CMakeLists.txt ---
@@ -14,14 +14,13 @@
# See the License for the specific language governing permissions and
# limitations under th
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/403
Zookeeper 2923
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JiangJiafu/zookeeper ZOOKEEPER-2923
Alternatively you can review and apply
GitHub user JiangJiafu opened a pull request:
https://github.com/apache/zookeeper/pull/408
ZOOKEEPER-2923
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JiangJiafu/zookeeper ZOOKEEPER-2923
Alternatively you can review and apply
Github user JiangJiafu closed the pull request at:
https://github.com/apache/zookeeper/pull/403
---
Github user asfgit closed the pull request at:
https://github.com/apache/zookeeper/pull/408
---
Github user Shonminh closed the pull request at:
https://github.com/apache/zookeeper/pull/380
---
GitHub user MichaelYgZhang opened a pull request:
https://github.com/apache/zookeeper/pull/442
Update zookeeperOver.html
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MichaelYgZhang/zookeeper master
Alternatively you can revie
Github user MichaelYgZhang closed the pull request at:
https://github.com/apache/zookeeper/pull/442
---
Github user 20100507 closed the pull request at:
https://github.com/apache/zookeeper/pull/376
---
GitHub user seunghoo opened a pull request:
https://github.com/apache/zookeeper/pull/460
missing list
In ZooKeeper.java, `existWatches.remove(clientPath;` removes watches for
clientPath and next `existWatches.remove(clientPath)` returns anything.
You can merge this pull request int
Github user seunghoo closed the pull request at:
https://github.com/apache/zookeeper/pull/460
---
GitHub user achimbab opened a pull request:
https://github.com/apache/zookeeper/pull/461
missing list
At line 518, 'existWatches.remove(clientPath)' is null because watches for
clientPath is already removed.
You can merge this pull request into a Git repository by running:
$ g
GitHub user cxxjava opened a pull request:
https://github.com/apache/zookeeper/pull/480
c++ zookeeper
c++ zookeeper base java zookeeper v3.4.11
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/cxxjava/zookeeper CxxZookeeper
Altern
Github user andschwa commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/386#discussion_r173568056
--- Diff: src/c/CMakeLists.txt ---
@@ -14,14 +14,13 @@
# See the License for the specific language governing permissions and
# limitations under
Github user andschwa commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/386#discussion_r173569107
--- Diff: src/c/CMakeLists.txt ---
@@ -14,14 +14,13 @@
# See the License for the specific language governing permissions and
# limitations under
Github user andschwa commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/386#discussion_r173571281
--- Diff: src/c/CMakeLists.txt ---
@@ -14,14 +14,13 @@
# See the License for the specific language governing permissions and
# limitations under
GitHub user brettKK opened a pull request:
https://github.com/apache/zookeeper/pull/494
Branch 3.5
callee BinaryInputArchive#startVector will return null:
```
// code placeholder
public Index startVector(String tag) throws IOException {
int len = readInt(tag);
GitHub user Rumo-Arf opened a pull request:
https://github.com/apache/zookeeper/pull/81
Update Watcher.java
Switch case clause, may wrote like this. Isn't it?
The other switch case clause wrote like that so...
You can merge this pull request into a Git repository by running
GitHub user fairliang opened a pull request:
https://github.com/apache/zookeeper/pull/86
zookeeper pull
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/fairliang/zookeeper trunk
Alternatively you can review and apply these chang
Github user fairliang closed the pull request at:
https://github.com/apache/zookeeper/pull/86
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
GitHub user rakeshadr opened a pull request:
https://github.com/apache/zookeeper/pull/98
ZOOKEEPER-2479
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rakeshadr/zookeeper-1 ZK-2479
Alternatively you can review and apply these c
Github user eribeiro commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/98#discussion_r86415195
--- Diff: src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java
---
@@ -520,6 +520,12 @@ public synchronized void setCurrentVote(Vote v){
Github user rakeshadr commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/98#discussion_r86483860
--- Diff: src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java
---
@@ -520,6 +520,12 @@ public synchronized void setCurrentVote(Vote v){
Github user eribeiro commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/98#discussion_r86502042
--- Diff: src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java
---
@@ -520,6 +520,12 @@ public synchronized void setCurrentVote(Vote v){
GitHub user rakeshadr opened a pull request:
https://github.com/apache/zookeeper/pull/101
ZOOKEEPER-2383
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rakeshadr/zookeeper-1 ZK-2383
Alternatively you can review and apply these
GitHub user benjiazhen opened a pull request:
https://github.com/apache/zookeeper/pull/107
Branch 3.3
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/zookeeper branch-3.3
Alternatively you can review and apply these chang
1 - 100 of 3719 matches
Mail list logo