gianm commented on PR #15093: URL: https://github.com/apache/druid/pull/15093#issuecomment-1748963581
This change is important but we need to do it less disruptively. People are using `MV_TO_ARRAY` because they need it for grouping on MVs during ingestion, and our docs recommend using it there. It's a relatively common pattern, and all these people would start getting string arrays instead of MVDs, which will break their queries. For this reason the change needs to be opt-in. I suggest documenting the context parameter and swapping the default so the old behavior is retained. In addition I'd suggest writing up a doc page about how people can migrate from MVDs to string arrays (with examples of how to rewrite queries) and pointing people at that in the docs for this parameter. That sets us up for a future where we can deprecate the parameter and swap the default. -- 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]
