You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Zoltan Haindrich <ki...@rxd.hu> on 2016/07/16 09:20:05 UTC
Re: Review Request 50104: HIVE-12244 Refactoring code for avoiding of comparison of Strings and do comparison on Path

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

(Updated July 16, 2016, 9:20 a.m.)


Review request for hive and Ashutosh Chauhan.


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


Repository: hive-git


Description
-------

keep Old prefixed methods of original methods for @Explain backward compat; small cleanups; add convinience methods for working with path2aliases and path2partitiondescs


Diffs
-----

  common/src/java/org/apache/hadoop/hive/common/FileUtils.java d755798 
  llap-server/src/java/org/apache/hadoop/hive/llap/io/api/impl/LlapInputFormat.java cc4e10b 
  ql/src/java/org/apache/hadoop/hive/ql/exec/AbstractMapOperator.java 5c3012b 
  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java bb43950 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MapOperator.java fabfbc4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 973bc23 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java 56f16cf 
  ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkDynamicPartitionPruner.java b70be01 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java a1e4e6c 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DynamicPartitionPruner.java 9e1bf0b 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/SplitGrouper.java b678884 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorMapOperator.java 0f55c00 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizedRowBatchCtx.java e06349d 
  ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexedInputFormat.java e072ee6 
  ql/src/java/org/apache/hadoop/hive/ql/io/BucketizedHiveInputFormat.java a9c1614 
  ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveInputFormat.java 49b03a4 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveContextAwareRecordReader.java 4a05a62 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java 6bb5efa 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java 227a051 
  ql/src/java/org/apache/hadoop/hive/ql/io/IOPrepareCache.java f4a21f8 
  ql/src/java/org/apache/hadoop/hive/ql/io/SymbolicInputFormat.java 8b49204 
  ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroGenericRecordReader.java 30862c8 
  ql/src/java/org/apache/hadoop/hive/ql/io/merge/MergeFileWork.java 3339c8d 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/ProjectionPusher.java db923fa 
  ql/src/java/org/apache/hadoop/hive/ql/io/rcfile/stats/PartialScanWork.java c006743 
  ql/src/java/org/apache/hadoop/hive/ql/io/rcfile/truncate/ColumnTruncateWork.java d63aa29 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java f555741 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/MapJoinProcessor.java e577e19 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/AbstractJoinTaskDispatcher.java a3a7f42 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/CommonJoinTaskDispatcher.java 9ea0857 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/GenMRSkewJoinProcessor.java f41fa4e 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/GenSparkSkewJoinProcessor.java ded9231 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/LlapDecider.java 4d50c64 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/NullScanTaskDispatcher.java 0b82596 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SortMergeJoinTaskDispatcher.java 658717c 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java bce3853 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkSkewJoinProcFactory.java 5990d17 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverCommonJoin.java 0d04e84 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java 3f07ea7 
  ql/src/java/org/apache/hadoop/hive/ql/plan/MapWork.java 8d329d9 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PartitionDesc.java fe09bdf 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 71dfc50 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestOperators.java 2ccb05a 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestPlan.java 06e5e07 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestUtilities.java 69f5465 
  ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestTezTask.java dc39cbd 
  ql/src/test/org/apache/hadoop/hive/ql/io/TestCombineHiveInputFormat.java 9b8a519 
  ql/src/test/org/apache/hadoop/hive/ql/io/TestHiveBinarySearchRecordReader.java 9dc4f5b 
  ql/src/test/org/apache/hadoop/hive/ql/io/TestHiveFileFormatUtils.java efc18ee 
  ql/src/test/org/apache/hadoop/hive/ql/io/TestSymlinkTextInputFormat.java 08b8c32 
  ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java 3c89dd6 
  ql/src/test/org/apache/hadoop/hive/ql/plan/TestConditionalResolverCommonJoin.java ef846a6 
  ql/src/test/org/apache/hadoop/hive/ql/plan/TestMapWork.java PRE-CREATION 

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


Testing
-------


Thanks,

Zoltan Haindrich