----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24195/ -----------------------------------------------------------
(Updated Aug. 1, 2014, 11:45 p.m.) Review request for hive. Bugs: HIVE-7561 https://issues.apache.org/jira/browse/HIVE-7561 Repository: hive-git Description ------- Hive uses the "assert" keyword all over the place. The problem is that assertions are rarely enabled since they have to be specifically enabled. In the Spark code, e.g. GenSparkUtils, let's use Preconditions.*. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkReduceSinkMapJoinProc.java 8c58333 ql/src/java/org/apache/hadoop/hive/ql/parse/spark/GenSparkUtils.java 25eea14 ql/src/java/org/apache/hadoop/hive/ql/parse/spark/GenSparkWork.java ceb7b6c ql/src/java/org/apache/hadoop/hive/ql/parse/spark/SparkProcessAnalyzeTable.java 3a0f4c9 ql/src/java/org/apache/hadoop/hive/ql/plan/SparkWork.java 86d14f1 Diff: https://reviews.apache.org/r/24195/diff/ Testing ------- Thanks, Chao Sun