Hive
  1. Hive
  2. HIVE-4387

ant maven-build fails because hcatalog doesn't have a make-pom target

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.11.0
    • Component/s: Build Infrastructure
    • Labels:
      None

      Description

      Other maven-* target may fail as well.

      1. HIVE-4387-branch-0.11.patch
        5 kB
        Alan Gates
      2. HIVE-4387-2.patch
        4 kB
        Alan Gates
      3. HIVE-4387.patch
        4 kB
        Alan Gates

        Issue Links

          Activity

          Hide
          Alan Gates added a comment -

          I patch to make maven-build and maven-publish work with HCatalog. I wasn't able to test maven-publish in the non-local case since I don't have permissions to push to the repos. Whoever reviews this needs to test that functionality.

          Show
          Alan Gates added a comment - I patch to make maven-build and maven-publish work with HCatalog. I wasn't able to test maven-publish in the non-local case since I don't have permissions to push to the repos. Whoever reviews this needs to test that functionality.
          Hide
          Ashutosh Chauhan added a comment -

          Patch is not applying cleanly on trunk.

          Show
          Ashutosh Chauhan added a comment - Patch is not applying cleanly on trunk.
          Hide
          Alan Gates added a comment -

          New version of the patch that applies to trunk.

          Show
          Alan Gates added a comment - New version of the patch that applies to trunk.
          Hide
          Ashutosh Chauhan added a comment -

          Patch applies cleanly but HCatalog jars are getting published in hcatalog namespace instead of hive namespace in local maven repo.

          Show
          Ashutosh Chauhan added a comment - Patch applies cleanly but HCatalog jars are getting published in hcatalog namespace instead of hive namespace in local maven repo.
          Hide
          Alan Gates added a comment -

          I think we should fix this when we move hcatalog into org.apache.hive. Otherwise we'll have the rather strange situation that classes in the namespace org.apache.hcatalog will be stored in maven in org.apache.hive. We will also break anyone's pom's that are depending on the existing jars.

          As part of moving code into org.apache.hive I can create a pom for org.apache.hcatalog that pull in the new org.apache.hive.hcatalog jars.

          Show
          Alan Gates added a comment - I think we should fix this when we move hcatalog into org.apache.hive. Otherwise we'll have the rather strange situation that classes in the namespace org.apache.hcatalog will be stored in maven in org.apache.hive. We will also break anyone's pom's that are depending on the existing jars. As part of moving code into org.apache.hive I can create a pom for org.apache.hcatalog that pull in the new org.apache.hive.hcatalog jars.
          Hide
          Ashutosh Chauhan added a comment -

          Ok. Make sense. Than I will only include this in 0.11 branch.

          Show
          Ashutosh Chauhan added a comment - Ok. Make sense. Than I will only include this in 0.11 branch.
          Hide
          Ashutosh Chauhan added a comment -

          Patch looks good. +1
          Doesn't apply cleanly on 0.11 though. Can you generate 0.11 version for it?

          Show
          Ashutosh Chauhan added a comment - Patch looks good. +1 Doesn't apply cleanly on 0.11 though. Can you generate 0.11 version for it?
          Hide
          Alan Gates added a comment -

          A version of the patch for branch 0.11.

          Show
          Alan Gates added a comment - A version of the patch for branch 0.11.
          Hide
          Ashutosh Chauhan added a comment -

          Committed to 0.11. Thanks, Alan!

          Show
          Ashutosh Chauhan added a comment - Committed to 0.11. Thanks, Alan!

            People

            • Assignee:
              Alan Gates
              Reporter:
              Alan Gates
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development