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