[fix][fe] Inconsistent behavior for string comparison in FE and BE

nextdreamblue · nextdreamblue · commit b2cc94a04fb9 · 2022-10-25T11:30:36.000+08:00
Signed-off-by: nextdreamblue &lt;zxw520blue1@163.com&gt;
diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java
@@ -96,9 +96,9 @@ public int compareLiteral(LiteralExpr expr) {
         int minLength = Math.min(thisBytes.length, otherBytes.length);
         int i = 0;
         for (i = 0; i < minLength; i++) {
-            if (thisBytes[i] < otherBytes[i]) {
+            if (Byte.toUnsignedInt(thisBytes[i]) < Byte.toUnsignedInt(otherBytes[i])) {
                 return -1;
-            } else if (thisBytes[i] > otherBytes[i]) {
+            } else if (Byte.toUnsignedInt(thisBytes[i]) > Byte.toUnsignedInt(otherBytes[i])) {
                 return 1;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -96,9 +96,9 @@ public int compareLiteral(LiteralExpr expr) {`
`96`	`96`	`int minLength = Math.min(thisBytes.length, otherBytes.length);`
`97`	`97`	`int i = 0;`
`98`	`98`	`for (i = 0; i < minLength; i++) {`
`99`		`- if (thisBytes[i] < otherBytes[i]) {`
	`99`	`+ if (Byte.toUnsignedInt(thisBytes[i]) < Byte.toUnsignedInt(otherBytes[i])) {`
`100`	`100`	`return -1;`
`101`		`- } else if (thisBytes[i] > otherBytes[i]) {`
	`101`	`+ } else if (Byte.toUnsignedInt(thisBytes[i]) > Byte.toUnsignedInt(otherBytes[i])) {`
`102`	`102`	`return 1;`
`103`	`103`	`}`
`104`	`104`	`}`