Uploaded image for project: 'Kafka'
  1. Kafka
  2. KAFKA-2167

ZkUtils updateEphemeralPath JavaDoc (spelling and correctness)

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • None
    • None

    Description

      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.

        /**
         * 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
          }
        }
      

      should be:

        /**
         * 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
          }
        }
      

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              jonbringhurst Jon Bringhurst
              Neha Narkhede Neha Narkhede
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: