abhishekrb19 commented on PR #14378: URL: https://github.com/apache/druid/pull/14378#issuecomment-1580156786
> I'm wondering why is not the `IS_AGGREGATOR` declared as `BOOLEAN`? > > IMO for boolean type, it would be more simplier and intuitive for user to filter out aggregator functions like > > ```sql > select * from ROUTINES where IS_AGGREGATOR > ``` Good question, @FrankChen021. I wondered the same :) The `INFORMATION_SCHEMA` tables in Druid already have boolean values [converted to varchar](https://github.com/apache/druid/blob/master/sql/src/main/java/org/apache/druid/sql/calcite/schema/InformationSchema.java#L112) - for example, `IS_JOINABLE`, `IS_BROADCAST`, `IS_NULLABLE`. The [SQL spec](http://web.cecs.pdx.edu/~len/sql1999.pdf) notes that, so most standard implementations are just following the spec. So for consistency, I made new column `VARCHAR` as it doesn't deviate too much from the spec. -- 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]
