morningman commented on a change in pull request #4658:
URL: https://github.com/apache/incubator-doris/pull/4658#discussion_r495464553
##########
File path:
fe/fe-core/src/main/java/org/apache/doris/load/loadv2/LoadingTaskPlanner.java
##########
@@ -182,6 +182,13 @@ public String getTimezone() {
}
}
+ // If this is a dynamic partitioned table, it will take some time to
create the partition after the
+ // table is created, a exception needs to be thrown here
+ if (partitionIds.isEmpty()) {
+ throw new LoadException("data cannot be inserted into table with
emtpy partition. " +
Review comment:
I agree. I think we can change the error msg like:
```
data cannot be inserted into table with emtpy partition... Use `SHOW
PARTITIONS FROM tbl` to see the currenty partitions of this table.
```
And also change the msg in error code `ERR_EMPTY_PARTITION_IN_TABLE`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
