Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/160#discussion_r98967876
--- Diff: src/java/main/org/apache/zookeeper/server/DataTree.java ---
@@ -1045,18 +1046,16 @@ private void getCounts(String path, Counts counts
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/150#discussion_r98263409
--- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java
---
@@ -87,15 +75,104 @@ public
StaticHostProvider(Collection
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/150#discussion_r98483994
--- Diff: src/java/main/org/apache/zookeeper/client/StaticHostProvider.java
---
@@ -122,18 +122,19 @@ public int size() {
private int
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/180#discussion_r101841044
--- Diff:
src/java/main/org/apache/zookeeper/server/command/SnapCommand.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/179#discussion_r101586810
--- Diff:
src/java/main/org/apache/zookeeper/server/command/FourLetterCommands.java ---
@@ -153,13 +155,33 @@
*/
public final
Github user edwardoliveira commented on the issue:
https://github.com/apache/zookeeper/pull/85
Thanks @hanm ! :smiley:
---
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
Github user edwardoliveira commented on the issue:
https://github.com/apache/zookeeper/pull/562
We could print an error message before quitting. I think -1 is a nice value
to return, but it's up to you. :)
Also, Zeus isn't the ZooKeeper fork from Facebook? Lol.
---
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/501#discussion_r181134958
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java ---
@@ -55,6 +53,7 @@
private final File snapDir
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/501#discussion_r181126991
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java ---
@@ -30,10 +30,8 @@
import
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/501#discussion_r181129644
--- Diff: src/java/main/org/apache/zookeeper/server/ServerStats.java ---
@@ -148,9 +150,23 @@ synchronized public void resetRequestCounters
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/501#discussion_r181127170
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java ---
@@ -135,6 +136,8 @@
long currentSize;
File
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/501#discussion_r181127430
--- Diff:
src/java/main/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java ---
@@ -55,6 +53,7 @@
private final File snapDir
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/500#discussion_r182180406
--- Diff: src/java/test/org/apache/zookeeper/server/quorum/Zab1_0Test.java
---
@@ -161,8 +127,8 @@ public void testLeaderInConnectingFollowers
GitHub user edwardoliveira opened a pull request:
https://github.com/apache/zookeeper/pull/493
ZOOKEEPER-3006: Potential NPE in ZKDatabase#calculateTxnLogSizeLimit
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/eribeiro
Github user edwardoliveira commented on the issue:
https://github.com/apache/zookeeper/pull/493
Hey, @LJ1043041006, I have just added a unit test. Cheers!
---
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/632#discussion_r223546118
--- Diff:
zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java ---
@@ -1521,4 +1562,179 @@ public boolean removeWatch(String
Github user edwardoliveira commented on the issue:
https://github.com/apache/zookeeper/pull/617
+1
---
17 matches
Mail list logo