Hive
  1. Hive
  2. HIVE-60

Fix the build scripts for the new hive tree

    Details

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

      Description

      Fix hive builds to reflect the hive subproject branch structure.

      1. patch-60.txt
        2.54 MB
        Ashish Thusoo

        Activity

        Hide
        David Phillips added a comment -

        I checked HBase and Pig and they both include the Hadoop jar, so I guess that's the accepted way. Seems like we only need a single Hadoop jar in lib?

        svn mv hadoopcore/lib/hadoop-0.20.0-dev-core.jar lib/
        svn rm hadoopcore

        Show
        David Phillips added a comment - I checked HBase and Pig and they both include the Hadoop jar, so I guess that's the accepted way. Seems like we only need a single Hadoop jar in lib? svn mv hadoopcore/lib/hadoop-0.20.0-dev-core.jar lib/ svn rm hadoopcore
        Hide
        Ashish Thusoo added a comment -

        You can point to an existing hadoop build using the -Dhadoop.jar to point to the location of the hadoop jar also setting hadoop.bin.path and hadoop.config.dir while running the tests.

        We decided to bundle hadoop jars (similar to what hbase and pig have done) in order to make it easy to check out hive and test it without having to build and deploy hadoop as well.

        However, we are going to move away from this model and use ivy to handle the dependencies and then all these jars will go away.

        Show
        Ashish Thusoo added a comment - You can point to an existing hadoop build using the -Dhadoop.jar to point to the location of the hadoop jar also setting hadoop.bin.path and hadoop.config.dir while running the tests. We decided to bundle hadoop jars (similar to what hbase and pig have done) in order to make it easy to check out hive and test it without having to build and deploy hadoop as well. However, we are going to move away from this model and use ivy to handle the dependencies and then all these jars will go away.
        Hide
        David Phillips added a comment -

        Why are we including a snapshot Hadoop build? Shouldn't we use HADOOP_HOME?

        Show
        David Phillips added a comment - Why are we including a snapshot Hadoop build? Shouldn't we use HADOOP_HOME?
        Hide
        Ashish Thusoo added a comment -

        This was checked in by Zheng.

        Show
        Ashish Thusoo added a comment - This was checked in by Zheng.
        Hide
        Prasad Chakka added a comment -

        +1
        reviewed manually. looks good for now.

        though we need to move to a directory structure that is similar to that of hadoop. (hive/trunk/src/ql hive/trunk/src/metastore)

        Show
        Prasad Chakka added a comment - +1 reviewed manually. looks good for now. though we need to move to a directory structure that is similar to that of hadoop. (hive/trunk/src/ql hive/trunk/src/metastore)
        Hide
        Ashish Thusoo added a comment -

        patch that fixes the build issues with the new tree.

        Show
        Ashish Thusoo added a comment - patch that fixes the build issues with the new tree.

          People

          • Assignee:
            Ashish Thusoo
            Reporter:
            Ashish Thusoo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development