Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r170009123
--- Diff: src/java/main/org/apache/zookeeper/server/OldEphemeralType.java
---
@@ -0,0 +1,74 @@
+/**
+ * Licensed to the Apache Software
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r169985373
--- Diff: src/java/main/org/apache/zookeeper/server/EphemeralType.java ---
@@ -37,41 +77,152 @@
/**
* TTL node
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r169984867
--- Diff: src/java/main/org/apache/zookeeper/server/OldEphemeralType.java
---
@@ -0,0 +1,74 @@
+/**
+ * Licensed to the Apache Software
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
FYI - I just pushed a change that adds yet-another-flag that allows 3.5.4
ZKs to read the old 3.5.3 TTL nodes. I think we must have this. The docs are
updated too.
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
IMPORTANT NOTE: TTL Nodes created in 3.5.3 will revert to EPHEMERAL with
this change. We need to discuss the impact of this and consider workarounds,
etc.
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/378
Attn: @phunt - I just pushed two changes:
Better docs and new reserved bits in the EphemeralType enum.
Better implementation of the testable serverId in ZooKeeperServer.
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
Attn: @phunt - I just pushed two changes:
- Better docs and new reserved bits in the `EphemeralType` enum.
- Better implementation of the testable `serverId` in ZooKeeperServer.
---
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r163747537
--- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
@@ -476,9 +474,12 @@ public ZooKeeperServerListener
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r163746348
--- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
@@ -476,9 +474,12 @@ public ZooKeeperServerListener
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r163671234
--- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
@@ -476,9 +474,12 @@ public ZooKeeperServerListener
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r159091495
--- Diff: src/docs/src/documentation/content/xdocs/zookeeperAdmin.xml ---
@@ -949,14 +949,15 @@ server.3=zoo3:2888:3888
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r159091284
--- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
@@ -476,9 +474,12 @@ public ZooKeeperServerListener
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
Controlling TTL Nodes via zoo.cfg turned out to be untennable. There are
too many parts of the code that need to know about TTLs being enabled or not.
The previous PR had several holes relating
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
> I used the sample zoo.cfg file as my config and started via
bin/zkServer.sh - however I can still create a ttl based node even though
"ttlNodesEnabled=false" in the zoo.cfg. I
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
@phunt it's back to default false now. So, I hope this can be merged.
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
@phunt you reviewed this PR originally so I was hoping you'd merge. @rgs1
can you merge if Pat can't?
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
@phunt Can we get this merged please?
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
re: ACL and recursive watches...
ZooKeeper.exists() does not check ACLs so I assert that the current
behavior of this PR doesn't violate the current behavior. i.e. in ZooKeeper you
can
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
One of our devs, @alexbrasetvik, asked about ACLs and recursive watches. It
turns out this PR is not handling that and needs to. I'll submit support for
this soon.
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
This PR and #332 are ready. How can we get this merged? @skamille ?
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
This feature is ready to merge (along with #378)
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
The integration above doesn't seem to update, but the build passes now.
---
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r141521002
--- Diff: src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
---
@@ -84,6 +84,9 @@ public static long initializeNextSession(long id
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/377#discussion_r141521029
--- Diff: src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java ---
@@ -125,6 +125,8 @@
private final ZooKeeperServerListener listener
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/380
Is this some kind of spam?
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
As promised - here's the project with the benchmarking stuff:
https://github.com/Randgalt/zkbench
---
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
> does the new map in the watch structure break rolling upgrades
It would depend how you did it. If you upgraded all the ZK servers before
adding any persistent watches you'd be OK.
---
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r140815418
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -40,50 +40,110 @@
class WatchManager {
private static final
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
FYI - I did some local ad-hoc testing with server IDs 254/255 and the
config value true/false in zoo.cfg and everything works as expected.
---
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/378
[ZOOKEEPER-2903] Backport of ZOOKEEPER-2901 changes
See https://github.com/apache/zookeeper/pull/377
There was a major oversight when TTL nodes were implemented. The session ID
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/377
There are 4 tests with this line of code and it's failing because the
logger is not there (NullPointerException). Any ideas?
```
Layout layout
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/377
[ZOOKEEPER-2901] TTL Nodes don't work with Server IDs > 127
There was a major oversight when TTL nodes were implemented. The session ID
generator for each server is seeded with the configu
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
Another goal is feature parity with other consensus tools such as
etcd/consul. I added TTL nodes with this (and other) goals earlier in the year
(or was it last year?). Watches in consul
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
Per 1. I posted some performance numbers in the issue. There's a definite
hit but it's worth it in my view. We should discuss this.
Per 2. What this PR is aimed at is users of Curator's
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132592200
--- Diff: src/java/main/org/apache/zookeeper/ZooDefs.java ---
@@ -74,12 +74,16 @@
public final int createTTL = 21
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132579217
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -97,23 +102,44 @@ synchronized void removeWatcher(Watcher watcher
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/332
Port of ZOOKEEPER-1416 Persistent Recursive Watches
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Randgalt/zookeeper ZOOKEEPER-2871
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
@afine issues addressed
---
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 Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132034244
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -181,11 +206,18 @@ synchronized void dumpWatches(PrintWriter pwriter
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132034154
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -97,23 +102,44 @@ synchronized void removeWatcher(Watcher watcher
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132034008
--- Diff: src/java/main/org/apache/zookeeper/ZooDefs.java ---
@@ -74,12 +74,16 @@
public final int createTTL = 21
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r132027605
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -97,23 +102,44 @@ synchronized void removeWatcher(Watcher watcher
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
@hanm done
---
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
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/249#discussion_r119611508
--- Diff: src/docs/src/documentation/content/xdocs/zookeeperReconfig.xml ---
@@ -338,7 +338,10 @@
server.3=125.23.63.25:2782:2785:participant
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/248#discussion_r119179974
--- Diff:
src/java/test/org/apache/zookeeper/test/ReconfigExceptionTestCase.java ---
@@ -0,0 +1,122 @@
+/**
+ * Licensed to the Apache Software
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/249
[ZOOKEEPER-2779] Branch 3.5 backport
Branch 3.5 backport of https://github.com/apache/zookeeper/pull/248
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/248
[ZOOKEEPER-2779} Provide a means to disable setting of the Read Only ACL
for the reconfig node
Provide a means to disable setting of the Read Only ACL for the reconfig
node added in ZOOKEEPER
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/222#discussion_r110944401
--- Diff: src/java/main/org/apache/zookeeper/ZooKeeper.java ---
@@ -1325,6 +1325,23 @@ public synchronized void close() throws
InterruptedException
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
@stuhood - FYI - here's a new version of Curator's cache recipes
consolidated into one-recipe-to-rule-them-all that relies on this new
Persistent Recursive Watch implementation.
https
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
Thanks @eribeiro - at minimum it would be good to get feedback. This
feature will really help the community.
---
If your project is set up for it, you can reply to this email and have your
Github user Randgalt closed the pull request at:
https://github.com/apache/zookeeper/pull/192
---
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 Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/192
merged into `branch-3.5`
---
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 Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/193
ZOOKEEPER-2608: CLI option for TTL nodes
This PR adds support for TTL nodes to the CLI
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/192
ZOOKEEPER-2719: Pull changes for TTL Nodes into 3.5 branch
This PR cherry-picks changes for TTL nodes (ZOOKEEPER-2169) and the CLI for
it (ZOOKEEPER-2608) into branch-3.5.
You can merge
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/152
Do I need to create a new patch file or is that no longer necessary?
---
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
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/152
[ZOOKEEPER-2642] reconfig() is now named reconfigure() due to ZOOKEEPER-2642
Was #122
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user Randgalt closed the pull request at:
https://github.com/apache/zookeeper/pull/122
---
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 Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/122
OK - to be clear, I have PR #151 which applies the legacy API to
branch-3.5. Then, I'll rework _this_ PR (against master) to only have the Admin
API, right?
---
If your project is set up
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/122
Do we need a separate issue for the 3.6 change?
---
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
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/122
See #151
---
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
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/151
Applying ZOOKEEPER-2642 patch to a 3.5 base
New PR that applies ZOOKEEPER-2642 to `branch-3.5`
You can merge this pull request into a Git repository by running:
$ git pull https
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/122
Will do
---
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
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/122
> Are you OK with us removing the deprecated methods when 3.5 becomes GA?
No. I'd vote against that. Breaking changes should be in a 3.6 version.
That was the genesis of this issue to be
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
@stuhood I'll end up writing one for Apache Curator. The short answer is
that large ZK users end up with 100s of thousands or millions of watchers when
a few dozen would suffice. The use case
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93893162
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -40,47 +40,52 @@
class WatchManager {
private static final
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
> it is required to add a C client change in this patch too?
@eribeiro C client work is separate. I usually add a new Jira for them.
If/when this patch is accepted I'll
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/136
@eribeiro Have a look at the change to `PathIterator` I think this solves
your desire while not breaking implicit contracts.
---
If your project is set up for it, you can reply to this email
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886810
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -181,11 +206,19 @@ synchronized void dumpWatches(PrintWriter pwriter
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886806
--- Diff: src/java/main/org/apache/zookeeper/server/WatchManager.java ---
@@ -97,23 +102,44 @@ synchronized void removeWatcher(Watcher watcher
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886802
--- Diff: src/java/main/org/apache/zookeeper/server/PathIterator.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886789
--- Diff: src/java/main/org/apache/zookeeper/server/PathIterator.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886801
--- Diff: src/java/main/org/apache/zookeeper/server/PathIterator.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886805
--- Diff: src/java/main/org/apache/zookeeper/ZooKeeper.java ---
@@ -522,6 +566,15 @@ protected boolean removeWatches(Map<String,
Set> pathVsW
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886777
--- Diff: src/java/main/org/apache/zookeeper/server/PathIterator.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886440
--- Diff: src/java/test/org/apache/zookeeper/test/PathIteratorTest.java ---
@@ -0,0 +1,75 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/136#discussion_r93886426
--- Diff: src/java/main/org/apache/zookeeper/server/PathIterator.java ---
@@ -0,0 +1,53 @@
+/**
+ * Licensed to the Apache Software Foundation
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/136
[ZOOKEEPER-1416] Persistent Recursive Watch
Here is a completed implementation for a persistent, recursive watch
addition for ZK. These watches are set via a new method, `addPersistentWatch
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/122#discussion_r92597471
--- Diff: src/java/main/org/apache/zookeeper/ZooKeeper.java ---
@@ -2910,4 +2923,49 @@ private ClientCnxnSocket getClientCnxnSocket()
throws IOException
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/122#discussion_r92597402
--- Diff: src/java/main/org/apache/zookeeper/ZooKeeper.java ---
@@ -39,32 +39,10 @@
import org.apache.zookeeper.client.ZKClientConfig;
import
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/122
[ZOOKEEPER-2642] Resurrect the reconfig() methods that were in
ZooKeeper.java.
Resurrect the reconfig() methods that were in ZooKeeper.java. While 3.5.x
is an alpha there are clients
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r88091830
--- Diff:
src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java ---
@@ -543,7 +546,7 @@ protected void pRequest2Txn(int type, long zxid
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/84
OK - I've put the args into container classes. Let me know what you think.
---
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
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/82
I never had to write any scripts so I really can't say. It all "just
worked". But, then, Curator started as a git project.
---
If your project is set up for it, you can reply to
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/82
On Curator I always use the [] and it works. ¯\_(ã)_/¯
---
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
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/82
Yeah - in other projects a merge autocloses these. Dunno why this didn't.
---
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
Github user Randgalt closed the pull request at:
https://github.com/apache/zookeeper/pull/82
---
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 Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r87054533
--- Diff: src/java/test/org/apache/zookeeper/test/KeyAuthClientTest.java ---
@@ -0,0 +1,131 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r86868035
--- Diff: src/java/test/org/apache/zookeeper/test/KeyAuthClientTest.java ---
@@ -0,0 +1,131 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/84
latest patch added
---
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
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/84
@fpj any updates on this?
---
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 Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/84
@fpj I've addressed and pushed changes for all your comments.
---
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
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r85649028
--- Diff:
src/java/main/org/apache/zookeeper/server/auth/ProviderRegistry.java ---
@@ -31,7 +31,15 @@
private static boolean initialized
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r85649005
--- Diff: src/java/test/org/apache/zookeeper/test/KeyAuthClientTest.java ---
@@ -0,0 +1,131 @@
+/**
+ * Licensed to the Apache Software Foundation
Github user Randgalt commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/84#discussion_r85648967
--- Diff: src/docs/src/documentation/content/xdocs/zookeeperProgrammers.xml
---
@@ -1193,6 +1193,55 @@ authProvider.2=com.f.MyAuth2
only one
Github user Randgalt commented on the issue:
https://github.com/apache/zookeeper/pull/84
@fpj Any more thoughts on this Flavio?
---
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
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/84
[ZOOKEEPER-1525] Plumb ZooKeeperServer object into auth plugins
Based on patch work from
https://issues.apache.org/jira/browse/ZOOKEEPER-1525
Created ServerAuthenticationProvider which
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/82
[ZOOKEEPER-2169] Enable creation of nodes with TTLs
This patch takes advantage of 3.5's container support. Most of the work
needed to support TTLs is there already.
In order
Github user Randgalt closed the pull request at:
https://github.com/apache/zookeeper/pull/29
---
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 Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/34
ZOOKEEPER-2163 - update doc for container options to reflect correct
version of 3.5.1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user Randgalt opened a pull request:
https://github.com/apache/zookeeper/pull/33
ZOOKEEPER-2168 - first pass at C changes for containers
The C build doesn't compile on my Mac and it's been ages since I wrote any
C. So, this needs careful review.
You can merge this pull
1 - 100 of 103 matches
Mail list logo