Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-1376

zkServer.sh does not correctly check for $SERVER_JVMFLAGS

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3.3, 3.3.4
    • Fix Version/s: 3.3.7, 3.4.5
    • Component/s: scripts
    • Labels:
    • Hadoop Flags:
      Reviewed

      Description

      It will always include it even if not defined, although not much harm.

      if [ "x$SERVER_JVMFLAGS" ]
      then
      JVMFLAGS="$SERVER_JVMFLAGS $JVMFLAGS"
      fi

      should use the std idiom.

      1. ZOOKEEPER-1376.patch
        0.3 kB
        Skye Wanderman-Milne

        Issue Links

          Activity

          Hide
          skye Skye Wanderman-Milne added a comment -

          This is already fixed in trunk but as part of a new feature (see ZOOKEEPER-1161).

          Here's a patch that can be applied directly to 3.3.3 and 3.3.4.

          Show
          skye Skye Wanderman-Milne added a comment - This is already fixed in trunk but as part of a new feature (see ZOOKEEPER-1161 ). Here's a patch that can be applied directly to 3.3.3 and 3.3.4.
          Hide
          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12546108/ZOOKEEPER-1376.patch
          against trunk revision 1386496.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          -1 patch. The patch command could not apply the patch.

          Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/1185//console

          This message is automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12546108/ZOOKEEPER-1376.patch against trunk revision 1386496. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/1185//console This message is automatically generated.
          Hide
          henryr Henry Robinson added a comment -

          +1, patch looks good to me. I'll commit shortly to 3.3 and 3.4.

          Show
          henryr Henry Robinson added a comment - +1, patch looks good to me. I'll commit shortly to 3.3 and 3.4.
          Hide
          henryr Henry Robinson added a comment -

          I just committed this to 3.3 and 3.4. Thanks Skye!

          Show
          henryr Henry Robinson added a comment - I just committed this to 3.3 and 3.4. Thanks Skye!

            People

            • Assignee:
              skye Skye Wanderman-Milne
              Reporter:
              phunt Patrick Hunt
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development