Review Request 29733: HIVE-9319 : Cleanup Modified Files [Spark Branch]

2015-01-08 Thread Szehon Ho

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

Review request for hive and Xuefu Zhang.


Repository: hive-git


Description
---

Note that this limits cleanup to lines of code changed in spark-branch in the 
merge to trunk, not cleanup of all of the modified files, in order to reduce 
merge conflicts.


Diffs
-

  ql/src/java/org/apache/hadoop/hive/ql/Driver.java fa40082 
  ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java b25a639 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java ee42f4c 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinTableContainerSerDe.java
 abdb6af 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveKey.java 33aeda4 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 6f216c9 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/Optimizer.java a6d5c62 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/unionproc/UnionProcessor.java 
fec6822 
  ql/src/java/org/apache/hadoop/hive/ql/parse/MapReduceCompiler.java 1b6de64 
  ql/src/java/org/apache/hadoop/hive/ql/plan/BaseWork.java 1efbb12 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java 
4582678 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 076d2fa 
  shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java 
f1743ae 

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


Testing
---


Thanks,

Szehon Ho



Re: Review Request 29733: HIVE-9319 : Cleanup Modified Files [Spark Branch]

2015-01-08 Thread Xuefu Zhang

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

Ship it!


Ship It!

- Xuefu Zhang


On Jan. 9, 2015, 12:01 a.m., Szehon Ho wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/29733/
 ---
 
 (Updated Jan. 9, 2015, 12:01 a.m.)
 
 
 Review request for hive and Xuefu Zhang.
 
 
 Repository: hive-git
 
 
 Description
 ---
 
 Note that this limits cleanup to lines of code changed in spark-branch in the 
 merge to trunk, not cleanup of all of the modified files, in order to reduce 
 merge conflicts.
 
 
 Diffs
 -
 
   ql/src/java/org/apache/hadoop/hive/ql/Driver.java fa40082 
   ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java b25a639 
   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java ee42f4c 
   
 ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinTableContainerSerDe.java
  abdb6af 
   ql/src/java/org/apache/hadoop/hive/ql/io/HiveKey.java 33aeda4 
   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 6f216c9 
   ql/src/java/org/apache/hadoop/hive/ql/optimizer/Optimizer.java a6d5c62 
   
 ql/src/java/org/apache/hadoop/hive/ql/optimizer/unionproc/UnionProcessor.java 
 fec6822 
   ql/src/java/org/apache/hadoop/hive/ql/parse/MapReduceCompiler.java 1b6de64 
   ql/src/java/org/apache/hadoop/hive/ql/plan/BaseWork.java 1efbb12 
   
 ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java 
 4582678 
   ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 076d2fa 
   shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java 
 f1743ae 
 
 Diff: https://reviews.apache.org/r/29733/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Szehon Ho