Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-4387

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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.patch
        4 kB
        Alan Gates
      2. HIVE-4387-2.patch
        4 kB
        Alan Gates
      3. HIVE-4387-branch-0.11.patch
        5 kB
        Alan Gates

        Issue Links

          Activity

          Hide
          alangates 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
          alangates 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
          ashutoshc Ashutosh Chauhan added a comment -

          Patch is not applying cleanly on trunk.

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

          New version of the patch that applies to trunk.

          Show
          alangates Alan Gates added a comment - New version of the patch that applies to trunk.
          Hide
          ashutoshc 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
          ashutoshc 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
          alangates 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
          alangates 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
          ashutoshc Ashutosh Chauhan added a comment -

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

          Show
          ashutoshc Ashutosh Chauhan added a comment - Ok. Make sense. Than I will only include this in 0.11 branch.
          Hide
          ashutoshc 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
          ashutoshc 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
          alangates Alan Gates added a comment -

          A version of the patch for branch 0.11.

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

          Committed to 0.11. Thanks, Alan!

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development