Fix small logic bug in JDBCEmitter in calculating when to truncate.

tballison · tballison · commit 48c934f54062 · 2024-05-06T11:15:58.000-04:00
diff --git a/tika-pipes/tika-emitters/tika-emitter-jdbc/src/main/java/org/apache/tika/pipes/emitter/jdbc/JDBCEmitter.java b/tika-pipes/tika-emitters/tika-emitter-jdbc/src/main/java/org/apache/tika/pipes/emitter/jdbc/JDBCEmitter.java
@@ -617,7 +617,7 @@ private static String getHandledTypes() {
     private static class StringNormalizer {
 
         String normalize(String emitKey, String columnName, String s, int maxLength) {
-            if (maxLength < 0 || s.length() < maxLength) {
+            if (maxLength < 0 || s.length() <= maxLength) {
                 return s;
             }
             LOGGER.warn("truncating {}->'{}' from {} chars to {} chars",

Original file line number	Diff line number	Diff line change
`@@ -617,7 +617,7 @@ private static String getHandledTypes() {`
`617`	`617`	`private static class StringNormalizer {`
`618`	`618`
`619`	`619`	`String normalize(String emitKey, String columnName, String s, int maxLength) {`
`620`		`- if (maxLength < 0 \|\| s.length() < maxLength) {`
	`620`	`+ if (maxLength < 0 \|\| s.length() <= maxLength) {`
`621`	`621`	`return s;`
`622`	`622`	`}`
`623`	`623`	`LOGGER.warn("truncating {}->'{}' from {} chars to {} chars",`