Github user vvysotskyi commented on a diff in the pull request:
https://github.com/apache/drill/pull/570#discussion_r160481004
--- Diff:
exec/java-exec/src/main/codegen/templates/StringOutputRecordWriter.java ---
@@ -146,7 +146,7 @@ public void writeField() throws IOException {
// TODO: error check
addField(fieldId, reader.readObject().toString());
- <#elseif minor.class == "VarChar" || minor.class == "Var16Char" ||
minor.class == "VarBinary">
+ <#elseif minor.class == "VarChar" || minor.class == "Var16Char" ||
minor.class == "VarBinary" || minor.class == "VarDecimal">
--- End diff --
May these types be moved into previous `elseif`?
---
