[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 ] Neelesh Srinivas Salian updated KAFKA-2167: --- Assignee: (was: Neelesh Srinivas Salian) > 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)
[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 ] Neelesh Srinivas Salian updated KAFKA-2167: --- Status: Open (was: Patch Available) > 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)
[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 ] Neelesh Srinivas Salian updated KAFKA-2167: --- Reviewer: Neha Narkhede [~nehanarkhede], could you please help review this JIRA? Thank you. Regards, Neelesh. 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 Fix For: 0.8.3 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)
[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 ] Neelesh Srinivas Salian updated KAFKA-2167: --- Fix Version/s: 0.8.2.0 Status: Patch Available (was: Open) Here is the pull request: https://github.com/apache/kafka/pull/64 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 Fix For: 0.8.2.0 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)
[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 ] Neelesh Srinivas Salian updated KAFKA-2167: --- Fix Version/s: (was: 0.8.2.0) 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 Fix For: 0.8.3 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)
[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 ] Jon Bringhurst updated KAFKA-2167: -- Labels: newbie (was: ) 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)