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

        Kevin Wilfong created issue -
        Kevin Wilfong made changes -
        Field Original Value New Value
        Attachment HIVE-2534.1.patch.txt [ 12501354 ]
        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
        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
        He Yongqiang added a comment -

        committed, thanks Kevin!

        Show
        He Yongqiang added a comment - committed, thanks Kevin!
        He Yongqiang made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ashutosh Chauhan made changes -
        Fix Version/s 0.9.0 [ 12317742 ]
        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
        Carl Steinbach made changes -
        Fix Version/s 0.8.0 [ 12316178 ]
        Carl Steinbach made changes -
        Fix Version/s 0.9.0 [ 12317742 ]
        Carl Steinbach made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 21h 53m 1 He Yongqiang 31/Oct/11 17:34
        Resolved Resolved Closed Closed
        46d 6h 22m 1 Carl Steinbach 16/Dec/11 23:56

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development