Uploaded image for project: 'Sqoop'
  1. Sqoop
  2. SQOOP-507

Update dist/pom.xml so that maven doesn't download tomcat every time

    Details

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

      Description

      I found it less ideal to download tomcat every time whenever I run mvn package -Pdist. We should add some simple logic to the pom file, so we don't have to download it unless it doesn't already exist in local workspace.

      1. SQOOP-507.patch
        0.8 kB
        Cheolsoo Park

        Activity

        Hide
        cheolsoo Cheolsoo Park added a comment -

        In fact, I found that logic is already in place. But since we always delete the tomcat tarball after untar it, it has to get downloaded again.

        Attached is the patch that removes the delete task from the dist profile.

        Show
        cheolsoo Cheolsoo Park added a comment - In fact, I found that logic is already in place. But since we always delete the tomcat tarball after untar it, it has to get downloaded again. Attached is the patch that removes the delete task from the dist profile.
        Hide
        bleeapache Bilung Lee added a comment -

        Look good to me. Patch is in. Thanks!

        Show
        bleeapache Bilung Lee added a comment - Look good to me. Patch is in. Thanks!

          People

          • Assignee:
            cheolsoo Cheolsoo Park
            Reporter:
            cheolsoo Cheolsoo Park
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development