hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Hunt" <>
Subject Review Request: HIVE-2117: insert overwrite ignoring partition location
Date Fri, 20 May 2011 16:25:07 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for hive and Carl Steinbach.


This change resolves a regression introduced by HIVE-1707, specifically that the partition
location (set via alter table partition location) is not being respected.

I addressed this by using the user specified location (as done originally), except in the
case with cross-filesystem moves (which was the concern in 1707).

This addresses bug HIVE-2117.


  ql/src/java/org/apache/hadoop/hive/ql/metadata/ bcacd35 
  ql/src/test/org/apache/hadoop/hive/ql/ PRE-CREATION 
  ql/src/test/org/apache/hadoop/hive/ql/ 06a0447 
  ql/src/test/org/apache/hadoop/hive/ql/ PRE-CREATION 
  ql/src/test/org/apache/hadoop/hive/ql/ 8c7c0b8 
  ql/src/test/queries/clientpositive/alter5.q PRE-CREATION 
  ql/src/test/results/clientpositive/alter5.q.out PRE-CREATION 



I added a new test which verifies partition location explicitly - as the existing tests ignore
this detail. This test failed w/o my fix applied, it passes with the fix applied.



  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message