clintropolis commented on issue #9707: URL: https://github.com/apache/druid/issues/9707#issuecomment-621602409
Ah yeah, @gianm is right, i haven't worked my way up to the SQL stuff yet. Go ahead if you've got a change in mind, I can work around whatever you get wired up here. I don't believe i'll be removing `getTypeName` or anything dramatic from the aggregators, at most just changing it to `getComplexTypeName` or something like that to make it clearer that after the change in my refactor that it's only real purpose is to map to a complex serde for writing/reading segments. Even after that change, it still seems useful for the information schema, I'd be happy to review your PR once it exists :+1: ---------------------------------------------------------------- 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]
