OIiveirra commented on code in PR #69032:
URL: https://github.com/apache/doris/pull/69032#discussion_r4369517404
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/MinBy.java:
##########
@@ -66,6 +66,11 @@ private MinBy(NullableAggregateFunctionParams
functionParams) {
@Override
public void checkLegalityBeforeTypeCoercion() {
+ for (Expression argument : getArguments()) {
+ if (argument.getDataType().isSpatialType()) {
Review Comment:
Fixed in 59fd5424679. MIN_BY/MAX_BY now recursively reject spatial ordering
keys, including state/rewrite paths. Scalar spatial result rejection is
unchanged; nested Array/Map/Struct results with ordinary keys remain supported.
Added nullable/typed-NULL, imported-array and endian coverage. FE tests and the
Iceberg regression passed.
--
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]
