----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38796/ -----------------------------------------------------------
(Updated Sept. 29, 2015, 9:23 a.m.) Review request for drill and Jinfeng Ni. Changes ------- Addressed review comments except the issue in HiveScan.getColumns not expanding the '\*'. Logged a separate bug DRILL-3852 as it is known issue. Repository: drill-git Description ------- Please jira DRILL-3209 for details. Diffs (updated) ----- contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/HivePartitionDescriptor.java 11c6455 contrib/storage-hive/core/src/main/java/org/apache/drill/exec/planner/sql/logical/ConvertHiveParquetScanToDrillParquetScan.java PRE-CREATION contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveDrillNativeParquetScan.java PRE-CREATION contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveDrillNativeParquetSubScan.java PRE-CREATION contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveDrillNativeScanBatchCreator.java PRE-CREATION contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveScan.java 9ada569 contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveStoragePlugin.java 23aa37f contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveSubScan.java 2181c2a contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/schema/DrillHiveTable.java b459ee4 contrib/storage-hive/core/src/test/java/org/apache/drill/exec/TestHivePartitionPruning.java f0b4bdc contrib/storage-hive/core/src/test/java/org/apache/drill/exec/TestHiveProjectPushDown.java 6423a36 contrib/storage-hive/core/src/test/java/org/apache/drill/exec/hive/TestHiveStorage.java 9211af6 contrib/storage-hive/core/src/test/java/org/apache/drill/exec/hive/TestInfoSchemaOnHiveStorage.java 6118be5 contrib/storage-hive/core/src/test/java/org/apache/drill/exec/store/hive/HiveTestDataGenerator.java 34a7ed6 exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java 66f9f03 exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java 5838bd1 Diff: https://reviews.apache.org/r/38796/diff/ Testing ------- Added unittests to test reading all supported types, project pushdown and partition pruning. Manually tested with Hive tables containing large amount of data (these tests will become part of the regression suite). Thanks, Venki Korukanti