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

ZkUtils updateEphemeralPath JavaDoc (spelling and correctness)

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      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

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: