Re: Review Request 66414: HIVE-17970 MM LOAD DATA with OVERWRITE doesn't use base_n directory concept

2018-04-20 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66414/ --- (Updated April 20, 2018, 11:15 p.m.) Review request for hive and Eugene

Re: Review Request 66414: HIVE-17970 MM LOAD DATA with OVERWRITE doesn't use base_n directory concept

2018-04-17 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66414/ --- (Updated April 18, 2018, 2:42 a.m.) Review request for hive and Eugene

Re: Review Request 66414: HIVE-17970 MM LOAD DATA with OVERWRITE doesn't use base_n directory concept

2018-04-17 Thread Sergey Shelukhin
> On April 17, 2018, 3:24 a.m., Eugene Koifman wrote: > > common/src/java/org/apache/hadoop/hive/common/JavaUtils.java > > Line 219 (original), 218 (patched) > > > > > > what is the purpose of this if block? seems

Re: Review Request 66414: HIVE-17970 MM LOAD DATA with OVERWRITE doesn't use base_n directory concept

2018-04-16 Thread Eugene Koifman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66414/#review201287 --- common/src/java/org/apache/hadoop/hive/common/JavaUtils.java

Re: Review Request 66414: HIVE-17970 MM LOAD DATA with OVERWRITE doesn't use base_n directory concept

2018-04-05 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66414/ --- (Updated April 5, 2018, 7:24 p.m.) Review request for hive and Eugene Koifman.