w41ter commented on code in PR #47200:
URL: https://github.com/apache/doris/pull/47200#discussion_r1921724314


##########
fe/fe-core/src/main/java/org/apache/doris/catalog/ListPartitionInfo.java:
##########
@@ -83,6 +83,12 @@ public PartitionItem 
createAndCheckPartitionItem(SinglePartitionDesc desc, boole
 
         // we might receive one whole empty values list, we should add default 
partition value for
         // such occasion
+
+        if (partitionKeyDesc.getInValues() == null) {
+            throw new DdlException(
+                    "Partitions types is invalid, expected In in list 
partitions");

Review Comment:
   Would you please describe the error message more clearly? I don't understand 
what the root cause of the error is if I only see this error message.
   
   By the way, perhaps checking in the analysis phase would be more suitable.



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to