You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Jun Rao (JIRA)" <ji...@apache.org> on 2015/09/02 16:37:46 UTC
[jira] [Updated] (KAFKA-2167) ZkUtils updateEphemeralPath JavaDoc
(spelling and correctness)
[ https://issues.apache.org/jira/browse/KAFKA-2167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jun Rao updated KAFKA-2167:
---------------------------
Fix Version/s: (was: 0.8.3)
> 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
> Assignee: Neelesh Srinivas Salian
> 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)