Github user mattyb149 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2638#discussion_r182100896
--- Diff:
nifi-commons/nifi-record/src/main/java/org/apache/nifi/serialization/record/ResultSetRecordSet.java
---
@@ -350,6 +350,8 @@ private static RecordFieldType getFieldType(final int
sqlType) {
return RecordFieldType.TIME;
case Types.TIMESTAMP:
case Types.TIMESTAMP_WITH_TIMEZONE:
+ case -101: // Oracle's TIMESTAMP WITH TIME ZONE
--- End diff --
@markap14 I don't think we need these cases (Calcite would never return
Oracle types, and there'd be no other way to generate them?) but put them in
just in case (as this does Java SQL Timestamp handling). If you agree then I
can remove this (although it's likely no-harm-no-foul)
---
