You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/04/09 19:49:28 UTC
[GitHub] [druid] himanshug commented on issue #9638: refactor internal type
system
himanshug commented on issue #9638: refactor internal type system
URL: https://github.com/apache/druid/pull/9638#issuecomment-611720548
> a lot of postaggs produce double typed arrays in their results, and in anticipation of what I think might be a good idea to do next which is merge ValueType and ExprType, I went ahead and added array types to the new and improved ValueType enumeration. ... This is perhaps controversial and worth discussion, but it also doesn't seem to introduce any ill effects as far as I can tell.
that looks OK to me
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org