jackylee-ch opened a new pull request, #827: URL: https://github.com/apache/paimon-rust/pull/827
Java's `FieldMinAggFactory` and `FieldMaxAggFactory` accept any type `TypeCheckUtils#isComparable` allows, which excludes only MAP, MULTISET, ROW, ARRAY, VECTOR, VARIANT and BLOB. `MinMaxState` covered eleven roots and omitted four that `InternalRowUtils#compare` handles: BOOLEAN (`Boolean.compare`), TIMESTAMP WITH LOCAL TIME ZONE (same branch as TIMESTAMP), BINARY and VARBINARY (`byteArrayCompare`). Configuring `max` on such a column made an aggregation table unreadable with `Aggregate function 'max' does not support data type ...`. The four are added. BLOB stays rejected even though Arrow maps it to the same `Binary`, and the accumulator now carries the Arrow timezone, read back from `paimon_type_to_arrow` so it cannot drift — a result array without it fails `RecordBatch::try_new` against the field. An outdated comment claimed BOOLEAN has no ordering; that test now pins BLOB. -- 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]
