ParquetWriters cleanup
@@ -15,7 +15,6 @@ package com.facebook.presto.parquet.writer;
import com.facebook.presto.common.type.CharType;
import com.facebook.presto.common.type.DecimalType;
-import com.facebook.presto.common.type.RealType;
import com.facebook.presto.common.type.Type;
import com.facebook.presto.common.type.VarbinaryType;
import com.facebook.presto.common.type.VarcharType;
@@ -47,6 +46,7 @@ import static com.facebook.presto.common.type.BooleanType.BOOLEAN;
import static com.facebook.presto.common.type.DateType.DATE;
import static com.facebook.presto.common.type.DoubleType.DOUBLE;
import static com.facebook.presto.common.type.IntegerType.INTEGER;
+import static com.facebook.presto.common.type.RealType.REAL;
import static com.facebook.presto.common.type.SmallintType.SMALLINT;
import static com.facebook.presto.common.type.TimestampType.TIMESTAMP;
import static com.facebook.presto.common.type.TinyintType.TINYINT;
@@ -151,7 +151,7 @@ class ParquetWriters
}
}
- private static PrimitiveValueWriter getValueWriter(ValuesWriter valuesWriter, com.facebook.presto.common.type.Type type, PrimitiveType parquetType)
+ private static PrimitiveValueWriter getValueWriter(ValuesWriter valuesWriter, Type type, PrimitiveType parquetType)
{
if (BOOLEAN.equals(type)) {
return new BooleanValueWriter(valuesWriter, parquetType);
@@ -171,12 +171,12 @@ class ParquetWriters
if (DOUBLE.equals(type)) {
return new DoubleValueWriter(valuesWriter, parquetType);
}
- if (RealType.REAL.equals(type)) {
+ if (REAL.equals(type)) {
return new RealValueWriter(valuesWriter, parquetType);
}
if (type instanceof VarcharType || type instanceof CharType || type instanceof VarbinaryType) {
return new CharValueWriter(valuesWriter, type, parquetType);
}
- throw new PrestoException(NOT_SUPPORTED, format("Unsupported type in parquet writer: %s", type));
+ throw new PrestoException(NOT_SUPPORTED, format("Unsupported type for Parquet writer: %s", type));
}
}
