github-actions[bot] commented on code in PR #63621:
URL: https://github.com/apache/doris/pull/63621#discussion_r3297782678
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/types/DataType.java:
##########
@@ -905,7 +905,12 @@ private static void validateCatalogDataType(Type
catalogType) {
if (catalogType.isMapType()) {
org.apache.doris.catalog.MapType mt =
(org.apache.doris.catalog.MapType) catalogType;
- validateNestedType(catalogType, mt.getKeyType());
+ Type mapKeyType = mt.getKeyType();
+ if (mapKeyType.isComplexType()) {
Review Comment:
This new rejection makes `CREATE TABLE ... MAP<ARRAY<INT>, STRING>` fail,
but `regression-test/suites/query_p0/show/test_nested_complex_switch.groovy`
still expects that DDL to succeed via `sql_m_a` (`test { sql sql_m_a }`). That
suite will now fail with `MAP key type must be a primitive type`. Please update
or remove the existing success case, or convert it to an expected-error case,
so the regression suite matches the new intended behavior.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
