Hive
  1. Hive
  2. HIVE-2534

HiveIndexResult creation fails due to file system issue

    Details

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

      Description

      If the file system for temp files differs from the default file system in the conf, HiveIndexResult's constructor fails.

        Activity

        Hide
        Hudson added a comment -

        Integrated in Hive-trunk-h0.21 #1048 (See https://builds.apache.org/job/Hive-trunk-h0.21/1048/)
        HIVE-2534: HiveIndexResult creation fails due to file system issue (Kevin Wilfong via He Yongqiang)

        heyongqiang : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1195580
        Files :

        • /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java
        Show
        Hudson added a comment - Integrated in Hive-trunk-h0.21 #1048 (See https://builds.apache.org/job/Hive-trunk-h0.21/1048/ ) HIVE-2534 : HiveIndexResult creation fails due to file system issue (Kevin Wilfong via He Yongqiang) heyongqiang : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1195580 Files : /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java
        Hide
        He Yongqiang added a comment -

        committed, thanks Kevin!

        Show
        He Yongqiang added a comment - committed, thanks Kevin!
        Hide
        He Yongqiang added a comment -

        +1, will commit after tests pass

        Show
        He Yongqiang added a comment - +1, will commit after tests pass
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/2602/
        -----------------------------------------------------------

        Review request for hive, Yongqiang He and namit jain.

        Summary
        -------

        Fixes an issue with HiveIndexResult where the constructor would fail if the file system for temp directories differed from the default file system in the conf.

        Instead of getting the file system from the conf, it gets it from the paths.

        Thanks to Yongqiang for identifying the issue and suggesting the fix.

        This addresses bug HIVE-2534.
        https://issues.apache.org/jira/browse/HIVE-2534

        Diffs


        trunk/ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java 1183507

        Diff: https://reviews.apache.org/r/2602/diff

        Testing
        -------

        Verified it fixed a query that was failing.

        Thanks,

        Kevin

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2602/ ----------------------------------------------------------- Review request for hive, Yongqiang He and namit jain. Summary ------- Fixes an issue with HiveIndexResult where the constructor would fail if the file system for temp directories differed from the default file system in the conf. Instead of getting the file system from the conf, it gets it from the paths. Thanks to Yongqiang for identifying the issue and suggesting the fix. This addresses bug HIVE-2534 . https://issues.apache.org/jira/browse/HIVE-2534 Diffs trunk/ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java 1183507 Diff: https://reviews.apache.org/r/2602/diff Testing ------- Verified it fixed a query that was failing. Thanks, Kevin

          People

          • Assignee:
            Kevin Wilfong
            Reporter:
            Kevin Wilfong
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development