Jon Bringhurst created KAFKA-2167:
-------------------------------------

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


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)

Reply via email to