You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Andrew Musselman (JIRA)" <ji...@apache.org> on 2016/10/17 18:24:58 UTC

[jira] [Commented] (KAFKA-2167) ZkUtils updateEphemeralPath JavaDoc (spelling and correctness)

    [ https://issues.apache.org/jira/browse/KAFKA-2167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15583009#comment-15583009 ] 

Andrew Musselman commented on KAFKA-2167:
-----------------------------------------

Looks like this ticket should be closed.

> ZkUtils updateEphemeralPath JavaDoc (spelling and correctness)
> --------------------------------------------------------------
>
>                 Key: KAFKA-2167
>                 URL: https://issues.apache.org/jira/browse/KAFKA-2167
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Jon Bringhurst
>              Labels: newbie
>
> I'm not 100% sure on this, but it seems like "persistent" should instead say "ephemeral" in the JavaDoc. Also, note that "parrent" is misspelled.
> {noformat}
>   /**
>    * Update the value of a persistent node with the given path and data.
>    * create parrent directory if necessary. Never throw NodeExistException.
>    */
>   def updateEphemeralPath(client: ZkClient, path: String, data: String): Unit = {
>     try {
>       client.writeData(path, data)
>     }
>     catch {
>       case e: ZkNoNodeException => {
>         createParentPath(client, path)
>         client.createEphemeral(path, data)
>       }
>       case e2 => throw e2
>     }
>   }
> {noformat}
> should be:
> {noformat}
>   /**
>    * Update the value of an ephemeral node with the given path and data.
>    * create parent directory if necessary. Never throw NodeExistException.
>    */
>   def updateEphemeralPath(client: ZkClient, path: String, data: String): Unit = {
>     try {
>       client.writeData(path, data)
>     }
>     catch {
>       case e: ZkNoNodeException => {
>         createParentPath(client, path)
>         client.createEphemeral(path, data)
>       }
>       case e2 => throw e2
>     }
>   }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)