----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29787/#review67603 -----------------------------------------------------------
pom.xml <https://reviews.apache.org/r/29787/#comment111661> A followup to get rid of this? ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java <https://reviews.apache.org/r/29787/#comment111662> We should restrain from using * in imports. - Xuefu Zhang On Jan. 9, 2015, 11:55 p.m., Szehon Ho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29787/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2015, 11:55 p.m.) > > > Review request for hive. > > > Bugs: HIVE-9257 > https://issues.apache.org/jira/browse/HIVE-9257 > > > Repository: hive-git > > > Description > ------- > > As the entire patch is too big, this shows the modified files. These have > been cleanuped as part of HIVE-9319, HIVE-9306, HIVE-9305. > > The new files can be found here: > http://svn.apache.org/repos/asf/hive/branches/spark/ or > https://github.com/apache/hive/tree/spark under: > # data/conf/spark/ > # > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithLocalClusterSpark.java > # > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestMultiSessionsHS2WithLocalClusterSpark.java > # itests/qtest-spark/ > # > ql/src/java/org/apache/hadoop/hive/ql/exec/SparkHashTableSinkOperator.java > # ql/src/java/org/apache/hadoop/hive/ql/exec/spark/ > # ql/src/java/org/apache/hadoop/hive/ql/lib/TypeRule.java > # > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SparkMapJoinProcessor.java > # > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/GenSparkSkewJoinProcessor.java > # > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SparkCrossProductCheck.java > # > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SparkMapJoinResolver.java > # ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/ > # ql/src/java/org/apache/hadoop/hive/ql/parse/spark/ > # > ql/src/java/org/apache/hadoop/hive/ql/plan/SparkBucketMapJoinContext.java > # ql/src/java/org/apache/hadoop/hive/ql/plan/SparkEdgeProperty.java > # ql/src/java/org/apache/hadoop/hive/ql/plan/SparkHashTableSinkDesc.java > # ql/src/java/org/apache/hadoop/hive/ql/plan/SparkWork.java > # > ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsAggregatorSpark.java > # ql/src/test/org/apache/hadoop/hive/ql/exec/spark/ > # ql/src/test/queries/clientpositive/auto_join_stats.q > # ql/src/test/queries/clientpositive/auto_join_stats2.q > # ql/src/test/queries/clientpositive/bucket_map_join_spark1.q > # ql/src/test/queries/clientpositive/bucket_map_join_spark2.q > # ql/src/test/queries/clientpositive/bucket_map_join_spark3.q > # ql/src/test/queries/clientpositive/bucket_map_join_spark4.q > # ql/src/test/queries/clientpositive/multi_insert_mixed.q > # ql/src/test/queries/clientpositive/multi_insert_union_src.q > # ql/src/test/queries/clientpositive/parallel_join0.q > # ql/src/test/queries/clientpositive/parallel_join1.q > # ql/src/test/queries/clientpositive/spark_test.q > # ql/src/test/queries/clientpositive/udf_example_add.q > # ql/src/test/results/clientpositive/auto_join_stats.q.out > # ql/src/test/results/clientpositive/auto_join_stats2.q.out > # ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out > # ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out > # ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out > # ql/src/test/results/clientpositive/bucket_map_join_spark4.q.out > # ql/src/test/results/clientpositive/multi_insert_mixed.q.out > # ql/src/test/results/clientpositive/multi_insert_union_src.q.out > # ql/src/test/results/clientpositive/parallel_join0.q.out > # ql/src/test/results/clientpositive/parallel_join1.q.out > # ql/src/test/results/clientpositive/spark/ > # ql/src/test/results/clientpositive/spark_test.q.out > # ql/src/test/results/clientpositive/udf_example_add.q.out > # spark-client/ > Cleanup and review of those have been done as part of HIVE-9281 and HIVE-9288. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java cd4beeb > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 8264b16 > data/conf/hive-log4j.properties a5b9c9a > itests/hive-unit/pom.xml f9f59c9 > itests/pom.xml 0a154d6 > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 878202a > pom.xml efe5e3a > ql/pom.xml 84e912e > ql/src/java/org/apache/hadoop/hive/ql/Context.java 0373273 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java 8bb6d0f > ql/src/java/org/apache/hadoop/hive/ql/HashTableLoaderFactory.java 10ad933 > ql/src/java/org/apache/hadoop/hive/ql/exec/ExplainTask.java 3f5f475 > ql/src/java/org/apache/hadoop/hive/ql/exec/FilterOperator.java b96faa4 > ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableSinkOperator.java > f0e04e7 > ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java 561bf3b > ql/src/java/org/apache/hadoop/hive/ql/exec/ScriptOperator.java 87eaa8f > ql/src/java/org/apache/hadoop/hive/ql/exec/TaskFactory.java e3469a0 > ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java ed03bb9 > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecMapperContext.java > 8b92f32 > > ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinEagerRowContainer.java > 65bb1b7 > > ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinTableContainerSerDe.java > da36848 > ql/src/java/org/apache/hadoop/hive/ql/io/HiveContextAwareRecordReader.java > fd79832 > ql/src/java/org/apache/hadoop/hive/ql/io/IOContext.java ac47975 > ql/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java 4e2b130 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/AbstractBucketJoinProc.java > 70c23a6 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/AbstractSMBJoinProc.java > c9e8086 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/BucketJoinProcCtx.java > 911b1c8 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/BucketMapjoinProc.java > 1260c83 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 9a74e1e > ql/src/java/org/apache/hadoop/hive/ql/optimizer/MapJoinProcessor.java > ccb3ce5 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/Optimizer.java 95b678b > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/ReduceSinkDeDuplication.java > 6a7733c > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/CrossProductCheck.java > 7ea0166 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java > c8e6ef5 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/unionproc/UnionProcessor.java > c973d98 > ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompilerFactory.java > af8e1da > ql/src/java/org/apache/hadoop/hive/ql/plan/BaseWork.java 41264f8 > ql/src/java/org/apache/hadoop/hive/ql/plan/BucketMapJoinContext.java > e3cab05 > > ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java > 298a465 > ql/src/java/org/apache/hadoop/hive/ql/plan/MapredLocalWork.java d566087 > ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceSinkDesc.java 57beb69 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java ac2cc86 > ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsAggregator.java > 027bd39 > ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsPublisher.java > 0b648aa > ql/src/test/org/apache/hadoop/hive/ql/exec/TestOperators.java 9359a7a > > ql/src/test/org/apache/hadoop/hive/ql/io/TestHiveBinarySearchRecordReader.java > f36f8e0 > ql/src/test/queries/clientpositive/disable_merge_for_bucketing.q 471d296 > ql/src/test/queries/clientpositive/merge1.q c7249af > ql/src/test/queries/clientpositive/merge2.q bb86dc2 > ql/src/test/queries/clientpositive/skewjoin_union_remove_1.q 164dca3 > ql/src/test/queries/clientpositive/skewjoin_union_remove_2.q 9a09849 > ql/src/test/queries/clientpositive/union_remove_1.q c87b3fe > ql/src/test/queries/clientpositive/union_remove_10.q 6701952 > ql/src/test/queries/clientpositive/union_remove_11.q 4b2fa42 > ql/src/test/queries/clientpositive/union_remove_12.q 69d0d0a > ql/src/test/queries/clientpositive/union_remove_13.q 7605f0e > ql/src/test/queries/clientpositive/union_remove_14.q a4fdfc8 > ql/src/test/queries/clientpositive/union_remove_15.q e3c937b > ql/src/test/queries/clientpositive/union_remove_16.q 537078b > ql/src/test/queries/clientpositive/union_remove_17.q d70f3d3 > ql/src/test/queries/clientpositive/union_remove_18.q 6352bc3 > ql/src/test/queries/clientpositive/union_remove_19.q 8d75afa > ql/src/test/queries/clientpositive/union_remove_2.q 83cd288 > ql/src/test/queries/clientpositive/union_remove_20.q f80f7c1 > ql/src/test/queries/clientpositive/union_remove_21.q 8963c25 > ql/src/test/queries/clientpositive/union_remove_22.q b0c1ccd > ql/src/test/queries/clientpositive/union_remove_23.q a1b989a > ql/src/test/queries/clientpositive/union_remove_24.q ec561e0 > ql/src/test/queries/clientpositive/union_remove_25.q 76c1ff5 > ql/src/test/queries/clientpositive/union_remove_3.q 9617f73 > ql/src/test/queries/clientpositive/union_remove_4.q cae323b > ql/src/test/queries/clientpositive/union_remove_5.q 5df84e1 > ql/src/test/queries/clientpositive/union_remove_6.q bfce26d > ql/src/test/queries/clientpositive/union_remove_7.q 3a95674 > ql/src/test/queries/clientpositive/union_remove_8.q a83a43e > ql/src/test/queries/clientpositive/union_remove_9.q e71f6dd > serde/pom.xml 8c60b30 > service/src/java/org/apache/hive/service/server/HiveServer2.java 4dae247 > shims/0.20S/src/main/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java > f881a7a > shims/0.23/pom.xml 924fc1a > shims/0.23/src/main/java/org/apache/hadoop/hive/shims/Hadoop23Shims.java > f65d394 > shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java > 988692a > > Diff: https://reviews.apache.org/r/29787/diff/ > > > Testing > ------- > > > Thanks, > > Szehon Ho > >