Github user gardellajuanpablo commented on a diff in the pull request: https://github.com/apache/nifi/pull/2625#discussion_r180747757 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractDatabaseFetchProcessor.java --- @@ -428,33 +429,44 @@ protected static String getMaxValueFromRow(ResultSet resultSet, protected static String getLiteralByType(int type, String value, String databaseType) { // Format value based on column type. For example, strings and timestamps need to be quoted switch (type) { - // For string-represented values, put in single quotes - case CHAR: - case LONGNVARCHAR: - case LONGVARCHAR: - case NCHAR: - case NVARCHAR: - case VARCHAR: - case ROWID: - case DATE: - case TIME: - return "'" + value + "'"; - case TIMESTAMP: - if (!StringUtils.isEmpty(databaseType) && databaseType.contains("Oracle")) { - // For backwards compatibility, the type might be TIMESTAMP but the state value is in DATE format. This should be a one-time occurrence as the next maximum value - // should be stored as a full timestamp. Even so, check to see if the value is missing time-of-day information, and use the "date" coercion rather than the - // "timestamp" coercion in that case - if (value.matches("\\d{4}-\\d{2}-\\d{2}")) { - return "date '" + value + "'"; - } else { - return "timestamp '" + value + "'"; - } + // For string-represented values, put in single quotes --- End diff -- done
---