Patrick Hunt commented on ZOOKEEPER-809:

Looks good to me. python and java tests are a nice touch.

A few nits:

succesful -> successful
adition -> additional
any detail on the session uuid? format and valid characters? min/max length?
why is session expired 503? would 400 be better?

80char max
try not to reformat existing code unless you're changing something in that 
area. makes it hard to review and confuses "blame" information

> Improved REST Interface
> -----------------------
>                 Key: ZOOKEEPER-809
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-809
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: contrib
>            Reporter: Andrei Savu
>            Assignee: Andrei Savu
>         Attachments: SPEC.txt, SPEC.txt, ZOOKEEPER-809.patch
> I would like to extend the existing REST Interface to also support:
> * configuration
> * ephemeral znodes
> * watches - PubSubHubbub 
> * ACLs 
> * basic authentication
> I want to do this because when building web applications that talks directly 
> to ZooKeeper a REST API it's a lot easier to use (there is no protocol 
> mismatch) than an API that uses persistent connections. I plan to use the 
> improved version to build a web-based administrative interface. 

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