-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40976/
-----------------------------------------------------------
Review request for hive.
Bugs: HIVE-12595
https://issues.apache.org/jira/browse/HIVE-12595
Repository: hive-git
Description
-------
It seems like there are lot of usage of abstract Operator class where only
possibility is TableScanOp. This results in lot of uncertainty when that
parameter is passed in and is littered with instanceOf checks all over.
Diffs
-----
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRFileSink1.java dcdc9ba
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRProcContext.java 0da5790
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRTableScan1.java af0ac90
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 0cd7b62
ql/src/java/org/apache/hadoop/hive/ql/optimizer/MapJoinFactory.java 647f863
ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkSortMergeJoinFactory.java
aca0630
ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java 27d7276
ql/src/java/org/apache/hadoop/hive/ql/parse/spark/GenSparkUtils.java 40c23a5
ql/src/java/org/apache/hadoop/hive/ql/plan/MapWork.java 73e8f6d
ql/src/test/org/apache/hadoop/hive/ql/parse/TestGenTezWork.java d9ab9c0
Diff: https://reviews.apache.org/r/40976/diff/
Testing
-------
Refactoring patch. Regression suite suffice.
Thanks,
Ashutosh Chauhan
