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 2019/06/25 04:08:19 UTC
[GitHub] [incubator-druid] clintropolis opened a new pull request #7958:
Expr multi val selector fix
clintropolis opened a new pull request #7958: Expr multi val selector fix
URL: https://github.com/apache/incubator-druid/pull/7958
This PR fixes an omission in #7588 of coercing `ExprEval` that output array types back into a string list, which caused issues with using filters against expression virtual columns that produced an array output, as well as problems coercing output for scan queries in druid-sql.
The added test to `MultiValueDimensionTest.java` would fail with no matching results prior to the changes of this PR.
Additionally, added a few druid-sql tests using multi-value string dimensions as `varchar` to ensure things work as expected, 3 of which would have failed prior to this PR.
----------------------------------------------------------------
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