Cassandra
  1. Cassandra
  2. CASSANDRA-3476

Allow user to override jvm options picked by cassandra-env.sh

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Fix Version/s: 1.1.0
    • Component/s: None
    • Labels:

      Description

      currently user have $JVM_OPTS - they are used before automagically generated ops.

      1. patch-env
        0.4 kB
        Radim Kolar
      2. 3476.patch.txt
        0.6 kB
        paul cannon

        Activity

        Hide
        Radim Kolar added a comment -

        patch attached

        Show
        Radim Kolar added a comment - patch attached
        Hide
        paul cannon added a comment -

        +1, although as a nitpick I'd prefer the variable be named JVM_EXTRA_OPTS instead of JVM_OVERRIDE_OPTS, since its value won't actually override the other JVM_OPTS.

        Show
        paul cannon added a comment - +1, although as a nitpick I'd prefer the variable be named JVM_EXTRA_OPTS instead of JVM_OVERRIDE_OPTS , since its value won't actually override the other JVM_OPTS .
        Hide
        paul cannon added a comment -

        Adjusted patch according to my last comment.

        Show
        paul cannon added a comment - Adjusted patch according to my last comment.
        Hide
        paul cannon added a comment -

        +1

        Show
        paul cannon added a comment - +1
        Hide
        Jonathan Ellis added a comment -

        committed

        Show
        Jonathan Ellis added a comment - committed
        Hide
        Hudson added a comment -

        Integrated in Cassandra #1251 (See https://builds.apache.org/job/Cassandra/1251/)
        add support for JVM_EXTRA_OPTS env variable to cassandra-env.sh
        patch by Radim Kolar; reviewed by Paul Cannon for CASSANDRA-3476

        jbellis : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1212843
        Files :

        • /cassandra/trunk/conf/cassandra-env.sh
        Show
        Hudson added a comment - Integrated in Cassandra #1251 (See https://builds.apache.org/job/Cassandra/1251/ ) add support for JVM_EXTRA_OPTS env variable to cassandra-env.sh patch by Radim Kolar; reviewed by Paul Cannon for CASSANDRA-3476 jbellis : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1212843 Files : /cassandra/trunk/conf/cassandra-env.sh

          People

          • Assignee:
            Radim Kolar
            Reporter:
            Radim Kolar
            Reviewer:
            paul cannon
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development