Sqoop
  1. Sqoop
  2. SQOOP-989

Sqoop2: setenv.sh overrides value of JAVA_OPTS

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.99.1
    • Fix Version/s: 1.99.3
    • Component/s: sqoop2-dist
    • Labels:
      None

      Description

      setenv.sh right now has something like:

      JAVA_OPTS=-Dsqoop.config.dir=`dirname $0`/../conf
      

      This overrides the JAVA_OPTS which causes pain when sending custom JAVA_OPTS to catalina.sh

      We should fix that to only append to the existing JAVA_OPTS variable.

      1. SQOOP-989.1.patch
        0.8 kB
        Mark Grover

        Issue Links

          Activity

          Hide
          Jarek Jarcec Cecho added a comment -

          It's one line change, so I'm giving my +1 here.

          Show
          Jarek Jarcec Cecho added a comment - It's one line change, so I'm giving my +1 here.
          Show
          Jarek Jarcec Cecho added a comment - The patch is in: https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=commit;h=0a6fcce582258fce7e344bf9f2dc274d1ff99fd3 Thank you Mark! Jarcec
          Hide
          Mark Grover added a comment -

          Thanks, Jarcec!

          Show
          Mark Grover added a comment - Thanks, Jarcec!
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop200 #268 (See https://builds.apache.org/job/Sqoop2-hadoop200/268/)
          SQOOP-989: Sqoop2: setenv.sh overrides value of JAVA_OPTS (Revision 0a6fcce582258fce7e344bf9f2dc274d1ff99fd3)

          Result = SUCCESS
          jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=0a6fcce582258fce7e344bf9f2dc274d1ff99fd3
          Files :

          • dist/src/main/server/bin/setenv.sh
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop200 #268 (See https://builds.apache.org/job/Sqoop2-hadoop200/268/ ) SQOOP-989 : Sqoop2: setenv.sh overrides value of JAVA_OPTS (Revision 0a6fcce582258fce7e344bf9f2dc274d1ff99fd3) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=0a6fcce582258fce7e344bf9f2dc274d1ff99fd3 Files : dist/src/main/server/bin/setenv.sh
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop100 #269 (See https://builds.apache.org/job/Sqoop2-hadoop100/269/)
          SQOOP-989: Sqoop2: setenv.sh overrides value of JAVA_OPTS (Revision 0a6fcce582258fce7e344bf9f2dc274d1ff99fd3)

          Result = SUCCESS
          jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=0a6fcce582258fce7e344bf9f2dc274d1ff99fd3
          Files :

          • dist/src/main/server/bin/setenv.sh
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop100 #269 (See https://builds.apache.org/job/Sqoop2-hadoop100/269/ ) SQOOP-989 : Sqoop2: setenv.sh overrides value of JAVA_OPTS (Revision 0a6fcce582258fce7e344bf9f2dc274d1ff99fd3) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=0a6fcce582258fce7e344bf9f2dc274d1ff99fd3 Files : dist/src/main/server/bin/setenv.sh

            People

            • Assignee:
              Mark Grover
              Reporter:
              Mark Grover
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development