Github user szvasas commented on a diff in the pull request: https://github.com/apache/sqoop/pull/60#discussion_r238984324 --- Diff: src/test/org/apache/sqoop/hive/TestHiveTypesForAvroTypeMapping.java --- @@ -41,30 +44,49 @@ private final String hiveType; private final Schema schema; + private final SqoopOptions options; - @Parameters(name = "hiveType = {0}, schema = {1}") + @Parameters(name = "hiveType = {0}, schema = {1}, options = {2}") public static Iterable<? extends Object> parameters() { return Arrays.asList( - new Object[]{"BOOLEAN", Schema.create(Schema.Type.BOOLEAN)}, - new Object[]{"INT", Schema.create(Schema.Type.INT)}, - new Object[]{"BIGINT", Schema.create(Schema.Type.LONG)}, - new Object[]{"FLOAT", Schema.create(Schema.Type.FLOAT)}, - new Object[]{"DOUBLE", Schema.create(Schema.Type.DOUBLE)}, - new Object[]{"STRING", Schema.createEnum("ENUM", "doc", "namespce", new ArrayList<>())}, // Schema.Type.ENUM - new Object[]{"STRING", Schema.create(Schema.Type.STRING)}, - new Object[]{"BINARY", Schema.create(Schema.Type.BYTES)}, - new Object[]{"BINARY", Schema.createFixed("Fixed", "doc", "space", 1) } - //, new Object[]{"DECIMAL", Schema.create(Schema.Type.UNION).} + new Object[]{"BOOLEAN", Schema.create(Schema.Type.BOOLEAN), new SqoopOptions()}, --- End diff -- We could static import the create methods from Schema so that would make these lines shorter.
---