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

(Updated Aug. 13, 2014, 2:48 a.m.)


Review request for hive, Brock Noland and Szehon Ho.


Changes
-------

As step 1 of HIVE-7675, Clone the implementation of ExecMapper and remove 
static state.


Bugs: HIVE-7643
    https://issues.apache.org/jira/browse/HIVE-7643


Repository: hive-git


Description (updated)
-------

ExecMapper contain static states, static variable "done" for example. Spark 
executor may execute multi tasks concurrently, ExecMapper static state updated 
by one task would influence the logic of another task, which may lead to 
unpredictable result. 


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/spark/HiveMapFunction.java b18dcbf 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/spark/HiveMapFunctionResultList.java 
90be4d6 
  ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkRecordHandler.java 
PRE-CREATION 

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


Testing
-------


Thanks,

chengxiang li

Reply via email to