Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3184#discussion_r236940080
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive3-processors/src/main/java/org/apache/nifi/util/hive/HiveJdbcCommon.java
---
@@ -402,6 +409,9 @@ public static long convertToCsvStream(final ResultSet
rs, final OutputStream out
rowValues.add("");
}
break;
+ case SQLXML:
+
rowValues.add(StringEscapeUtils.escapeCsv(((java.sql.SQLXML)
value).getString()));
--- End diff --
Shouldn't we take care of null `value` here, too?
---
