[jira] Updated: (ZOOKEEPER-579) zkpython needs more test coverage for ACL code paths

2010-02-05 Thread Henry Robinson (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-579?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Henry Robinson updated ZOOKEEPER-579:
-

Attachment: zookeeper-579.patch

Patch adds test for aget_acl, aset_acl, set_acl and get_acl. 

Also adds tests in the C code for ACL objects that are roughly correctly 
formed, throwing InvalidACL where they are not found. This is to prevent an 
unpleasant bug where passing another object, say a callable, could silently 
succeed until a strange error condition would present. 

 zkpython needs more test coverage for ACL code paths
 

 Key: ZOOKEEPER-579
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-579
 Project: Zookeeper
  Issue Type: Improvement
  Components: contrib-bindings
Affects Versions: 3.2.1
Reporter: Henry Robinson
Assignee: Henry Robinson
 Attachments: zookeeper-579.patch


 zkpython's tests don't do a good enough job of exercising the ACL code paths. 
 A few new tests that confirm that setACL and friends are working correctly 
 are needed. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-321) optmize session tracking in zookeeper.

2010-02-05 Thread Flavio Paiva Junqueira (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12830194#action_12830194
 ] 

Flavio Paiva Junqueira commented on ZOOKEEPER-321:
--

In your example, it seems that we would be generating 256kbits/s, which is 
indeed small if we are talking about 100Mbit/1Gbit interfaces. Thanks for the 
insight. 

 optmize session tracking in zookeeper.
 --

 Key: ZOOKEEPER-321
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-321
 Project: Zookeeper
  Issue Type: New Feature
  Components: c client, java client, server
Reporter: Mahadev konar
Assignee: Mahadev konar
 Fix For: 3.3.0


 sometimes a lot of zookeeper clients are read only. For such clients we do 
 not need the session tracking in zookeeper. Getting rid of session tracking 
 for such clients will help us sclae much better.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-601) allow configuration of session timeout min/max bounds

2010-02-05 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-601?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-601:
---

Status: Patch Available  (was: Open)

 allow configuration of session timeout min/max bounds
 -

 Key: ZOOKEEPER-601
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-601
 Project: Zookeeper
  Issue Type: Improvement
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-601.patch


 ZK servers currently enforce a min/max boundary on client session timeout 
 relative to the ticktime setting, detailed here:
 http://hadoop.apache.org/zookeeper/docs/current/zookeeperProgrammers.html#ch_zkSessions
 In general there are good reasons for this however in some cases, in 
 particular with HBase region servers, we have seen a need to allow this bound 
 to be set differently (higher). The Sun jvm can GC pause for very long times 
 (in some cases we've seen 4 minutes even with the realtime gc.
 It would be good to allow this bound to be set via configuration parameters. 
 Note: 4letterword and JMX integration would be needed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-601) allow configuration of session timeout min/max bounds

2010-02-05 Thread Patrick Hunt (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-601?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Patrick Hunt updated ZOOKEEPER-601:
---

Attachment: ZOOKEEPER-601.patch

ZOOKEEPER-595 and ZOOKEEPER-617 are also addressed in this patch

1) the min/max session timeout can now be configured for a server (conf file/ 
jmx support enabled)
2) the conf 4letter word has been added (595)
3) cleanup some forrest doc (617)

During review please note:

1) new file added, be sure to svn add it
2) LeaderZKServer was missing a getServerID override, added it
3) test the conf 4 letter word
4) test jmx, verify you can see the params
5) tests added for everything
6) be sure to regenerate the forrest docs and commit


 allow configuration of session timeout min/max bounds
 -

 Key: ZOOKEEPER-601
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-601
 Project: Zookeeper
  Issue Type: Improvement
  Components: server
Affects Versions: 3.2.1
Reporter: Patrick Hunt
Assignee: Patrick Hunt
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-601.patch


 ZK servers currently enforce a min/max boundary on client session timeout 
 relative to the ticktime setting, detailed here:
 http://hadoop.apache.org/zookeeper/docs/current/zookeeperProgrammers.html#ch_zkSessions
 In general there are good reasons for this however in some cases, in 
 particular with HBase region servers, we have seen a need to allow this bound 
 to be set differently (higher). The Sun jvm can GC pause for very long times 
 (in some cases we've seen 4 minutes even with the realtime gc.
 It would be good to allow this bound to be set via configuration parameters. 
 Note: 4letterword and JMX integration would be needed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (ZOOKEEPER-607) improve bookkeeper overview

2010-02-05 Thread Benjamin Reed (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12830405#action_12830405
 ] 

Benjamin Reed commented on ZOOKEEPER-607:
-

it looks good flavio just two suggestions:

1) i think it is too much information when you start talking about the naming 
convention for the znodes in zookeeper.
2) the motivation is good, but i don't think we really need to discussion the 
alternatives for hdfs.

 improve bookkeeper overview
 ---

 Key: ZOOKEEPER-607
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-607
 Project: Zookeeper
  Issue Type: Improvement
  Components: contrib-bookkeeper
Reporter: Benjamin Reed
 Attachments: ZOOKEEPER-607.patch, ZOOKEEPER-607.patch, 
 ZOOKEEPER-607.patch


 fix the overview section in the bookkeeper documentation to introduce the 
 programmer/admin to bookkeeper before giving the details.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-607) improve bookkeeper overview

2010-02-05 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-607?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-607:


Fix Version/s: 3.3.0

will be good to get this in 3.3

 improve bookkeeper overview
 ---

 Key: ZOOKEEPER-607
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-607
 Project: Zookeeper
  Issue Type: Improvement
  Components: contrib-bookkeeper
Reporter: Benjamin Reed
 Fix For: 3.3.0

 Attachments: ZOOKEEPER-607.patch, ZOOKEEPER-607.patch, 
 ZOOKEEPER-607.patch


 fix the overview section in the bookkeeper documentation to introduce the 
 programmer/admin to bookkeeper before giving the details.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.