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.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

          Eugene Koifman made changes -
          Link This issue is related to HIVE-4871 [ HIVE-4871 ]
          Amareshwari Sriramadasu made changes -
          Link This issue is related to HIVE-4710 [ HIVE-4710 ]
          Owen O'Malley made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Ashutosh Chauhan made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Ashutosh Chauhan added a comment -

          Committed to 0.11. Thanks, Alan!

          Show
          Ashutosh Chauhan added a comment - Committed to 0.11. Thanks, Alan!
          Alan Gates made changes -
          Attachment HIVE-4387-branch-0.11.patch [ 12580338 ]
          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 -

          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
          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
          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 -

          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.
          Alan Gates made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Alan Gates made changes -
          Attachment HIVE-4387-2.patch [ 12580067 ]
          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.
          Ashutosh Chauhan made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          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.
          Alan Gates made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Alan Gates made changes -
          Attachment HIVE-4387.patch [ 12579908 ]
          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.
          Alan Gates made changes -
          Description Other *-pom directives probably fail as well. Other maven-* target may fail as well.
          Alan Gates made changes -
          Field Original Value New Value
          Summary ant make-pom fails because hcatalog doesn't have a make-pom target ant maven-build fails because hcatalog doesn't have a make-pom target
          Alan Gates created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development