----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33006/ -----------------------------------------------------------
Review request for drill, Daniel Barclay, Hanifi Gunes, and Jason Altekruse. Repository: drill-git Description ------- See DRILL-2641 for details. Also: - Update the tests to use the new test framework - Remove duplicate tests (TestHiveScalarUDFs.java) - Remove JDBC module dependency on Hive storage plugin module. - Disabled few tests in jdbc.DatabaseMetaDataGetColumnsTest which were using Hive table for certails column types such as array, map, struct and union which are not possible as input types for filesystem tables. This test was blocking from removing exec/jdbc module dependency on contrib/storage-hive module. Diffs ----- contrib/storage-hive/core/src/main/resources/bootstrap-storage-plugins.json 5c7174e contrib/storage-hive/core/src/test/java/org/apache/drill/exec/hive/TestHiveStorage.java a76128f contrib/storage-hive/core/src/test/java/org/apache/drill/exec/hive/TestInfoSchemaOnHiveStorage.java PRE-CREATION contrib/storage-hive/core/src/test/java/org/apache/drill/exec/store/hive/HiveTestDataGenerator.java 657da61 exec/java-exec/src/test/java/org/apache/drill/exec/sql/TestInfoSchema.java PRE-CREATION exec/jdbc/pom.xml 2385d74 exec/jdbc/src/test/java/org/apache/drill/jdbc/DatabaseMetaDataGetColumnsTest.java 1a7b63d exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestHiveScalarUDFs.java 2c1738c exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestHiveStorage.java 2c41e12 exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestMetadataDDL.java 3534f35 exec/jdbc/src/test/resources/bootstrap-storage-plugins.json 548ec3d Diff: https://reviews.apache.org/r/33006/diff/ Testing ------- Thanks, Venki Korukanti
