oglego commented on PR #4376: URL: https://github.com/apache/datafusion-comet/pull/4376#issuecomment-4513631030
I dug into this a bit more, and the underlying cardinality support is coming through Size. Enabling MapType there does allow cardinality/size on existing map-typed inputs, but constructor cases like cardinality(map(...)) still fall back because [CreateMap](https://github.com/apache/datafusion-comet/issues/3167) is not supported yet. Given that, I’m going to hold off on developing this for now and wait until CreateMap support is in place so the feature boundary is cleaner and the map support story is more consistent. -- 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]
