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