[BugFix] fix unique table skip short circuit

fe/fe-core/src/main/java/com/starrocks/sql/ShortCircuitPlannerHybrid.java

@@ -43,7 +43,7 @@ public LogicalPlanChecker(boolean allowFilter, boolean allowLimit, boolean allow
         public Boolean visitLogicalTableScan(OptExpression optExpression, Void context) {
             LogicalScanOperator scanOp = optExpression.getOp().cast();
             Table table = scanOp.getTable();
-            if (!(table instanceof OlapTable) && !(((OlapTable) table).getKeysType().equals(KeysType.PRIMARY_KEYS))) {
+            if (!(table instanceof OlapTable) || !(KeysType.PRIMARY_KEYS.equals(((OlapTable) table).getKeysType()))) {
                 return false;
             }
 

test/sql/test_spill/R/test_short_circuit_unique

@@ -0,0 +1,17 @@
+-- name: test_short_circuit_unique
+create table unique_tbl(a int, b string) 
+unique key(a)
+distributed by hash(a)
+PROPERTIES('replication_num' = '1');
+-- result:
+-- !result
+insert into unique_tbl values(1,'hi'),(2,'hello');
+-- result:
+-- !result
+set global enable_short_circuit = true;
+-- result:
+-- !result
+select * from unique_tbl where a=1;
+-- result:
+1	hi
+-- !result

test/sql/test_spill/T/test_short_circuit_unique

@@ -0,0 +1,10 @@
+-- name: test_short_circuit_unique
+-- 
+-- for issue: https://github.com/StarRocks/starrocks/issues/23491
+create table unique_tbl(a int, b string) 
+unique key(a)
+distributed by hash(a)
+PROPERTIES('replication_num' = '1');
+insert into unique_tbl values(1,'hi'),(2,'hello');
+set global enable_short_circuit = true;
+select * from unique_tbl where a=1;