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().


---

Reply via email to