zerolbsony commented on code in PR #16741:
URL: https://github.com/apache/iotdb/pull/16741#discussion_r2517671964


##########
iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/parser/ASTVisitor.java:
##########
@@ -344,6 +344,9 @@ public class ASTVisitor extends 
IoTDBSqlParserBaseVisitor<Statement> {
   private static final String LIMIT_CONFIGURATION_ENABLED_ERROR_MSG =
       "Limit configuration is not enabled, please enable it first.";
 
+  public static final String DELETE_RANGE_COMPARISON_ERROR_MSG =
+      "For delete statement, where clause use a range comparison on the same 
field, the left value of the range cannot be greater than the right value of 
the range, it must be written like this : time > 5 and time < 10";

Review Comment:
   > No need to be so wordy, "Invalid deletion range: [%d, %d]" should be enough
   
   Actually I originally wish so, but change my mind after I saw some similar 
error message template exist.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to