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

Patrick Hunt commented on ZOOKEEPER-809:
----------------------------------------

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

A few nits:

spec:
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?

code:
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