Sqoop
  1. Sqoop
  2. SQOOP-990

Sqoop2: Allow users to override Sqoop2's default http port and admin port

    Details

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

      Description

      Sqoop2 presently doesn't provide an option for the user to override the tomcat http port and the admin port. We should create properties so that these ports can be overriden by specifiying properties like:

      export SQOOP_HTTP_PORT=12000
      export SQOOP_ADMIN_PORT=12001
      
      1. SQOOP-990.1.patch
        5 kB
        Mark Grover
      2. SQOOP-990.2.patch
        6 kB
        Mark Grover
      3. SQOOP-990.3.patch
        6 kB
        Mark Grover

        Issue Links

          Activity

          Hide
          Mark Grover added a comment -

          For the benefit of the readers:
          the default value of HTTP port is 12000 and that of admin port is 1 more than HTTP port (i.e. 12001).

          In order to override the values, the user can set environment variables like:

          export SQOOP_HTTP_PORT=12008
          export SQOOP_ADMIN_PORT=12020
          

          Alternatively, the following parameters can be sent to catalina.sh (SQOOP-982 allows to do this via sqoop.sh script)

          -Dsqoop.http.port=12008 -Dsqoop.admin.port=12020
          
          Show
          Mark Grover added a comment - For the benefit of the readers: the default value of HTTP port is 12000 and that of admin port is 1 more than HTTP port (i.e. 12001). In order to override the values, the user can set environment variables like: export SQOOP_HTTP_PORT=12008 export SQOOP_ADMIN_PORT=12020 Alternatively, the following parameters can be sent to catalina.sh ( SQOOP-982 allows to do this via sqoop.sh script) -Dsqoop.http.port=12008 -Dsqoop.admin.port=12020
          Hide
          Jarek Jarcec Cecho added a comment -

          Mark Grover, it seems that the patch is not applying cleanly on current head of sqoop2 branch. Would you mind rebase it?

          Show
          Jarek Jarcec Cecho added a comment - Mark Grover , it seems that the patch is not applying cleanly on current head of sqoop2 branch. Would you mind rebase it?
          Hide
          Mark Grover added a comment -

          Uploading a new patch rebased on trunk.

          Show
          Mark Grover added a comment - Uploading a new patch rebased on trunk.
          Hide
          Jarek Jarcec Cecho added a comment - - edited

          Hi Mark Grover,
          I do have couple of nits. Would you mind uploading your patch to the review board for easier discussion?

          Jarcec

          Show
          Jarek Jarcec Cecho added a comment - - edited Hi Mark Grover , I do have couple of nits. Would you mind uploading your patch to the review board for easier discussion? Jarcec
          Hide
          Mark Grover added a comment -
          Show
          Mark Grover added a comment - Sure thing! https://reviews.apache.org/r/10662/
          Hide
          Mark Grover added a comment - - edited

          Updated patch based on review feedback

          Show
          Mark Grover added a comment - - edited Updated patch based on review feedback
          Hide
          Jarek Jarcec Cecho added a comment -

          The patch is in: https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=shortlog;h=refs/heads/sqoop2

          Thank you Mark for your contribution!

          Jarcec

          Show
          Jarek Jarcec Cecho added a comment - The patch is in: https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=shortlog;h=refs/heads/sqoop2 Thank you Mark for your contribution! Jarcec
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop200 #271 (See https://builds.apache.org/job/Sqoop2-hadoop200/271/)
          SQOOP-990: Sqoop2: Allow users to override Sqoop2's default http port and admin port (Revision fbd238627f329ffee980365f7695c50fa1d9576b)

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

          • dist/src/main/bin/sqoop.sh
          • dist/src/main/server/bin/setenv.sh
          • dist/src/main/bin/sqoop-sys.sh
          • dist/src/main/server/conf/server.xml
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop200 #271 (See https://builds.apache.org/job/Sqoop2-hadoop200/271/ ) SQOOP-990 : Sqoop2: Allow users to override Sqoop2's default http port and admin port (Revision fbd238627f329ffee980365f7695c50fa1d9576b) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=fbd238627f329ffee980365f7695c50fa1d9576b Files : dist/src/main/bin/sqoop.sh dist/src/main/server/bin/setenv.sh dist/src/main/bin/sqoop-sys.sh dist/src/main/server/conf/server.xml
          Hide
          Hudson added a comment -

          Integrated in Sqoop2-hadoop100 #272 (See https://builds.apache.org/job/Sqoop2-hadoop100/272/)
          SQOOP-990: Sqoop2: Allow users to override Sqoop2's default http port and admin port (Revision fbd238627f329ffee980365f7695c50fa1d9576b)

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

          • dist/src/main/server/conf/server.xml
          • dist/src/main/bin/sqoop.sh
          • dist/src/main/bin/sqoop-sys.sh
          • dist/src/main/server/bin/setenv.sh
          Show
          Hudson added a comment - Integrated in Sqoop2-hadoop100 #272 (See https://builds.apache.org/job/Sqoop2-hadoop100/272/ ) SQOOP-990 : Sqoop2: Allow users to override Sqoop2's default http port and admin port (Revision fbd238627f329ffee980365f7695c50fa1d9576b) Result = SUCCESS jarcec : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=fbd238627f329ffee980365f7695c50fa1d9576b Files : dist/src/main/server/conf/server.xml dist/src/main/bin/sqoop.sh dist/src/main/bin/sqoop-sys.sh 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