You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Kevin Wilfong <ke...@fb.com> on 2011/10/28 21:45:45 UTC

Review Request: Fix HiveIndexResult's constructor's issue with file systems.

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