ychernysh commented on PR #2937: URL: https://github.com/apache/drill/pull/2937#issuecomment-2334210046
@paul-rogers Regarding default type and `comments` example above. Yes, as I said, unhardcoding the `NullableInt` and making the default minor type configurable by user might be a good idea, but here are some concerns here: 1. Wouldn't this conflict with defaulting to `UntypedNull`? 2. Since the vectors for missing columns are _null-filled_, they can be casted/converted to any minor type. That is, the user can issue `CAST` or `CONVERT_TO` and it should help for a query to succeed. Well, then, if we can define a default minor type on a per-query level, does it make sense to configure it on per-session or per-system level? Note that such workaround also helps with the `UNION ALL` example I above in my previous comment. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
