[
https://issues.apache.org/jira/browse/ZOOKEEPER-731?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13097286#comment-13097286
]
[email protected] commented on ZOOKEEPER-731:
---------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1715/#review1751
-----------------------------------------------------------
src/java/main/org/apache/zookeeper/ZooKeeper.java
<https://reviews.apache.org/r/1715/#comment3981>
it is unrelated to the rest of the patch, but I touched the file anyway and
it's an obvious clean up that follows the boy scout rule.[1]
The previous LOG initialization was very confusing. It made me think
whether there's some JVM magic that would require LOG to be initialized in the
static block rather then in the normal way.
[1] http://pragmaticcraftsman.com/2011/03/the-boy-scout-rule/
- Thomas
On 2011-09-05 15:55:47, Thomas Koch wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/1715/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-09-05 15:55:47)
bq.
bq.
bq. Review request for zookeeper.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. .
bq.
bq.
bq. This addresses bug ZOOKEEPER-731.
bq. https://issues.apache.org/jira/browse/ZOOKEEPER-731
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. src/java/main/org/apache/zookeeper/ZooKeeper.java 00bac9f
bq.
bq. Diff: https://reviews.apache.org/r/1715/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Thomas
bq.
bq.
> Zookeeper#delete , #create - async versions miss a verb in the javadoc
> ------------------------------------------------------------------------
>
> Key: ZOOKEEPER-731
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-731
> Project: ZooKeeper
> Issue Type: Bug
> Components: documentation
> Affects Versions: 3.3.0
> Reporter: Kay Kay
> Priority: Minor
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-731.patch
>
>
> /**
> * The Asynchronous version of delete. "The request doesn't *missing*
> actually until
> * the asynchronous callback is called."
> */
> public void delete(final String path, int version, VoidCallback cb,
> Object ctx) ..
> Also some information in the javadoc about how to instantiate the callback
> objects / context would be useful .
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
