[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-78?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697699#action_12697699
 ] 

Mahadev konar edited comment on ZOOKEEPER-78 at 4/9/09 6:14 PM:
----------------------------------------------------------------

thanks for looking at it ben and pat... i am fixing most of the stuff you guys 
mentioned except for the following -- 

- the zookeeper c client methods -
pat's idea does make sense for namespace conventions. But 
zoo_recipes_lock_methodname seems too long. I recommend we use 
zoo_recipename_methodname().. which is a little mroe convenient thatn the older 
one.

there are others I will prefer opening a  jira later to fix it.. (like 
interperability of c and java code).. i would like this patch to get committed 
as soon as possible. maintaing such a huge patch is quite a burden for me.

      was (Author: mahadev):
    thanks for looking at it ben and pat... i am fixing most of the stuff you 
guys mentioned except for the following -- 

- the zookeeper c client methods -
pat's idea does make sense for namespace conventions. But 
zoo_recipes_lock_methodname seems too long. I recommend we use 
zoo_recipename_methodname().. which is a little mroe convenient thatn the older 
one.

there are others I will prefer opening a  jira later to fix it.. (like 
interperability of c and java code).. i would like this patch to get committed 
as soon as possible. maintaing such a huge patch is quite a huge burden for me.
  
> added a high level protocol/feature - for easy Leader Election or exclusive 
> Write Lock creation
> -----------------------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-78
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-78
>             Project: Zookeeper
>          Issue Type: New Feature
>          Components: java client
>    Affects Versions: 3.0.0
>            Reporter: james strachan
>            Assignee: Mahadev konar
>             Fix For: 3.2.0
>
>         Attachments: patch_with_including_Benjamin's_fix.patch, 
> using_zookeeper_facade.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, 
> ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, ZOOKEEPER-78.patch, 
> ZOOKEEPER-78.patch, ZOOKEEPER-78.patch
>
>
> Here's a patch which adds a little WriteLock helper class for performing 
> leader elections or creating exclusive locks in some directory znode. Note 
> its an early cut; am sure we can improve it over time. The aim is to avoid 
> folks having to use the low level ZK stuff but provide a simpler high level 
> abstraction.

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

Reply via email to