Hive
  1. Hive
  2. HIVE-4535

hive build fails with hadoop 0.20

    Details

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

      Description

      ant package -Dhadoop.mr.rev=20
      leads to -

          [javac] /Users/thejas/hive_thejas_git/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java:382: cannot find symbol
          [javac] symbol  : method join(java.lang.String,java.util.List<java.lang.String>)
          [javac] location: class org.apache.hadoop.util.StringUtils
          [javac]           StringUtils.join(",", incompatibleCols)
      
      
      1. HIVE-4535.2.patch
        3 kB
        Thejas M Nair
      2. HIVE-4535.1.patch
        3 kB
        Thejas M Nair

        Activity

        Hide
        Ashutosh Chauhan added a comment -

        This issue has been fixed and released as part of 0.12 release. If you find further issues, please create a new jira and link it to this one.

        Show
        Ashutosh Chauhan added a comment - This issue has been fixed and released as part of 0.12 release. If you find further issues, please create a new jira and link it to this one.
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-h0.21 #2126 (See https://builds.apache.org/job/Hive-trunk-h0.21/2126/)
        HIVE-4535 : hive build fails with hadoop 0.20 (Thejas Nair via Ashutosh Chauhan) (Revision 1488739)

        Result = FAILURE
        hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1488739
        Files :

        • /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
        Show
        Hudson added a comment - Integrated in Hive-trunk-h0.21 #2126 (See https://builds.apache.org/job/Hive-trunk-h0.21/2126/ ) HIVE-4535 : hive build fails with hadoop 0.20 (Thejas Nair via Ashutosh Chauhan) (Revision 1488739) Result = FAILURE hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1488739 Files : /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-hadoop2 #223 (See https://builds.apache.org/job/Hive-trunk-hadoop2/223/)
        HIVE-4535 : hive build fails with hadoop 0.20 (Thejas Nair via Ashutosh Chauhan) (Revision 1488739)

        Result = ABORTED
        hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1488739
        Files :

        • /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
        Show
        Hudson added a comment - Integrated in Hive-trunk-hadoop2 #223 (See https://builds.apache.org/job/Hive-trunk-hadoop2/223/ ) HIVE-4535 : hive build fails with hadoop 0.20 (Thejas Nair via Ashutosh Chauhan) (Revision 1488739) Result = ABORTED hashutosh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1488739 Files : /hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
        Hide
        Ashutosh Chauhan added a comment -

        Committed to trunk. Thanks, Thejas!

        Show
        Ashutosh Chauhan added a comment - Committed to trunk. Thanks, Thejas!
        Hide
        Carl Steinbach added a comment -

        +1. I can commit it if you're willing to test it.

        Not directly related to this patch - I wonder if we should send the exception stack trace also through MetaException. It would make it more info to clients so that users can debug better. Ideally, MetaException should have supported a second 'detailed error message' string.

        This sounds like a really good idea to me.

        Show
        Carl Steinbach added a comment - +1. I can commit it if you're willing to test it. Not directly related to this patch - I wonder if we should send the exception stack trace also through MetaException. It would make it more info to clients so that users can debug better. Ideally, MetaException should have supported a second 'detailed error message' string. This sounds like a really good idea to me.
        Hide
        Thejas M Nair added a comment -

        Thanks for the review, Carl. Attaching new patch.

        Not directly related to this patch - I wonder if we should send the exception stack trace also through MetaException. It would make it more info to clients so that users can debug better. Ideally, MetaException should have supported a second 'detailed error message' string.

        Show
        Thejas M Nair added a comment - Thanks for the review, Carl. Attaching new patch. Not directly related to this patch - I wonder if we should send the exception stack trace also through MetaException. It would make it more info to clients so that users can debug better. Ideally, MetaException should have supported a second 'detailed error message' string.
        Hide
        Carl Steinbach added a comment -

        I left some comments on RB. Thanks.

        Show
        Carl Steinbach added a comment - I left some comments on RB. Thanks.
        Hide
        Thejas M Nair added a comment -
        Show
        Thejas M Nair added a comment - Review board link - https://reviews.apache.org/r/11036/
        Hide
        Thejas M Nair added a comment -

        Issue is only in trunk, not in 0.11 branch.

        Show
        Thejas M Nair added a comment - Issue is only in trunk, not in 0.11 branch.

          People

          • Assignee:
            Thejas M Nair
            Reporter:
            Thejas M Nair
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development