Github user mattyb149 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2625#discussion_r180768758
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractDatabaseFetchProcessor.java
---
@@ -436,13 +437,26 @@ protected static String getLiteralByType(int type,
String value, String database
case NVARCHAR:
case VARCHAR:
case ROWID:
- case DATE:
- case TIME:
return "'" + value + "'";
+ case TIME:
--- End diff --
See my comment below, perhaps we'd have three methods then,
getDateLiteral(), getTimeLiteral(), and getTimestampLiteral().
---
