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