[jira] Updated: (ZOOKEEPER-218) error in barrier recipe example code

["Patrick Hunt (JIRA)" <ji...@apache.org>]

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

Patrick Hunt updated ZOOKEEPER-218:
-----------------------------------

    Attachment: ZOOKEEPER-218.patch

it seemed to me that the arg to the method should be root, not "name", I've updated the code as such. Also made the node creation as eph/seq so that multiple processes per host can participate in a specific barrier


> error in barrier recipe example code
> ------------------------------------
>
>                 Key: ZOOKEEPER-218
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-218
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: documentation
>    Affects Versions: 3.0.0
>            Reporter: Patrick Hunt
>             Fix For: 3.0.1, 3.1.0
>
>         Attachments: ZOOKEEPER-218.patch
>
>
> Report from user Murali Vemulapati. The wiki recipe page also should be updated to handle multiple clients using the barrier on the same system (multiple processes, etc...). Currently the docs/example support only a single user of the barrier per host.
> ------------------
> I believe there is a typo in the barrier example given at:
> http://hadoop.apache.org/zookeeper/docs/current/zookeeperTutorial.html
> With the following fix, the program runs as expected:
> ==============
> 83c83
> <                 this.name = new String(InetAddress.getLocalHost().getCanonicalHostName().toString());
> ---
> >                 name = new String(InetAddress.getLocalHost().getCanonicalHostName().toString());
> 100c100
> <                     CreateMode.EPHEMERAL_SEQUENTIAL);
> ---
> >                     CreateMode.EPHEMERAL);
> ==============
> The first change assigns the name to the instance variable 'name' of Barrier class (otherwise the 'name' instance variable will have a value of 'null'
> when calling zk.create to create the child node under the root barrier node).
> The second change lets us run multiple processes on the same machine.
> thanks
> murali

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