Re: Apache now has reviewboard

2010-10-26 Thread Henry Robinson
Yes! On 25 October 2010 22:47, Patrick Hunt ph...@apache.org wrote: And we're on it: https://reviews.apache.org/groups/zookeeper/ https://reviews.apache.org/groups/zookeeper/We should rework our howtocommit to incorporate this. Patrick On Mon, Oct 25, 2010 at 10:16 PM, Patrick Hunt

[jira] Updated: (ZOOKEEPER-913) Version parser fails to parse 3.3.2-dev from build.xml.

2010-10-26 Thread Anthony Urso (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Urso updated ZOOKEEPER-913: --- Attachment: zk-version.patch zk-build.patch Two possible solutions.

Re: Allowing a ZooKeeper server to be part of multiple clusters

2010-10-26 Thread Flavio Junqueira
That proposal came in the context of federated zookeeper, and the motivation at the time was to use multiple overlapping clusters to enable increasing write throughput as we increase the number of servers. To my knowledge, we haven't made any progress on the implementation of such a feature.I'd be

[jira] Updated: (ZOOKEEPER-912) ZooKeeper client logs trace and debug messages at level INFO

2010-10-26 Thread Patrick Hunt (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-912?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-912: --- Fix Version/s: 3.4.0 Assignee: Anthony Urso Issue Type: Improvement (was:

[jira] Updated: (ZOOKEEPER-913) Version parser fails to parse 3.3.2-dev from build.xml.

2010-10-26 Thread Patrick Hunt (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-913: --- Fix Version/s: 3.4.0 Assignee: Anthony Urso I like the version patch better.

[jira] Updated: (ZOOKEEPER-884) Remove LedgerSequence references from BookKeeper documentation and comments in tests

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-884?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-884: Fix Version/s: 3.4.0 Remove LedgerSequence references from BookKeeper documentation

[jira] Updated: (ZOOKEEPER-907) Spurious KeeperErrorCode = Session moved messages

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-907?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-907: Status: Patch Available (was: Open) Spurious KeeperErrorCode = Session moved messages

[jira] Updated: (ZOOKEEPER-702) GSoC 2010: Failure Detector Model

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-702: Fix Version/s: 3.4.0 GSoC 2010: Failure Detector Model

[jira] Commented: (ZOOKEEPER-904) super digest is not actually acting as a full superuser

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12925087#action_12925087 ] Mahadev konar commented on ZOOKEEPER-904: - good catch. +1 for the patch. Ill

Re: Review Request: Lower log levels in ZK client for trace/debug messages.

2010-10-26 Thread Patrick Hunt
--- This is an automatically generated e-mail. To reply, visit: http://reviews.apache.org/r/7/#review2 --- Most of the messages you changed are things we rely on to post-mortem

[jira] Commented: (ZOOKEEPER-912) ZooKeeper client logs trace and debug messages at level INFO

2010-10-26 Thread Patrick Hunt (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-912?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12925145#action_12925145 ] Patrick Hunt commented on ZOOKEEPER-912: I have issues with this patch that I

Re: Apache now has reviewboard

2010-10-26 Thread Mahadev Konar
That¹s really nice. mahadev On 10/25/10 10:16 PM, Patrick Hunt ph...@apache.org wrote: FYI: https://blogs.apache.org/infra/entry/reviewboard_instance_running_at_the We should start using this, I've used it for other projects and it worked out quite well. Patrick

Re: Apache now has reviewboard

2010-10-26 Thread Patrick Hunt
FYI apache infra is working on things like email notifications and such (partially works). For the time being update the JIRA with a link if you create a RB request or comment. We should think about what we want to do going forward. For example changing zk's how to contribute to reflect new dev

[jira] Commented: (ZOOKEEPER-87) Follower does not shut itself down if its too far behind the leader.

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-87?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12925165#action_12925165 ] Mahadev konar commented on ZOOKEEPER-87: exactly vishal. The problem is to avoid

Re: New strategy for Netty (ZOOKEEPER-823) was: What's the QA strategy of ZooKeeper?

2010-10-26 Thread Mahadev Konar
This sounds great. Thanks mahadev On 10/16/10 1:56 AM, Thomas Koch tho...@koch.ro wrote: Benjamin Reed: actually, the other way of doing the netty patch (since i'm scared of merges) would be to do a refactor cleanup patch with an eye toward netty, and then another patch to actually add

Re: Review Request: testing rb please ignore

2010-10-26 Thread gavin
On 2010-10-26 14:09:52, Patrick Hunt wrote: test review comment test comment, lemme know if you get it! - admin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8/#review3

Re: Review Request: testing rb please ignore

2010-10-26 Thread gavin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8/ --- (Updated 2010-10-26 15:01:35.042001) Review request for zookeeper. Changes

Review Request: This is a test review, it doesnt say much

2010-10-26 Thread gavin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9/ --- Review request for zookeeper. Summary --- test review, please ignore Diffs

[jira] Commented: (ZOOKEEPER-904) super digest is not actually acting as a full superuser

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12925177#action_12925177 ] Mahadev konar commented on ZOOKEEPER-904: - Ran the tests, it passes. I just

[jira] Updated: (ZOOKEEPER-904) super digest is not actually acting as a full superuser

2010-10-26 Thread Mahadev konar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-904?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-904: Resolution: Fixed Hadoop Flags: [Reviewed] Status: Resolved (was: Patch

Review Request: ZOOKEEPER-912: Lower log levels in ZK client for trace/debug messages.

2010-10-26 Thread Patrick Hunt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7/ --- Review request for zookeeper. Summary (updated) --- Lower log levels in ZK

[jira] Commented: (ZOOKEEPER-706) large numbers of watches can cause session re-establishment to fail

2010-10-26 Thread Patrick Hunt (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-706?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12925222#action_12925222 ] Patrick Hunt commented on ZOOKEEPER-706: This is a pretty easy one for someone