Skip to content

Commit

Permalink
Merge pull request #315 from palantir/jt/fix-decimal
Browse files Browse the repository at this point in the history
[SPARK-23388][SQL] Support for Parquet Binary DecimalType in VectorizedColumnReader
  • Loading branch information
jamesthomp committed Feb 12, 2018
2 parents 7257b91 + 14033cd commit ef31055
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -446,7 +446,8 @@ private void readBinaryBatch(int rowId, int num, WritableColumnVector column) {
// This is where we implement support for the valid type conversions.
// TODO: implement remaining type conversions
VectorizedValuesReader data = (VectorizedValuesReader) dataColumn;
if (column.dataType() == DataTypes.StringType || column.dataType() == DataTypes.BinaryType) {
if (column.dataType() == DataTypes.StringType || column.dataType() == DataTypes.BinaryType
|| DecimalType.isByteArrayDecimalType(column.dataType())) {
defColumn.readBinarys(num, column, rowId, maxDefLevel, data);
} else if (column.dataType() == DataTypes.TimestampType) {
if (!shouldConvertTimestamps()) {
Expand Down

0 comments on commit ef31055

Please sign in to comment.