[PR] remove druid.sql.planner.serializeComplexValues config in favor of always serializing complex values (druid)

["clintropolis (via GitHub)" <gi...@apache.org>]

clintropolis opened a new pull request, #17549:
URL: https://github.com/apache/druid/pull/17549

   This config option was introduced back in #6950 mainly as an escape hatch to make things backwards compatible, but setting it to false to get the old behavior is basically useless since it causes any complex type query results to just print the java class name instead of serializing the actual value into a usable form. As such, this config was never documented because the new behavior was superior, so should be totally safe to remove.


-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

Re: [PR] remove druid.sql.planner.serializeComplexValues config in favor of always serializing complex values (druid)

["clintropolis (via GitHub)" <gi...@apache.org>]

clintropolis merged PR #17549:
URL: https://github.com/apache/druid/pull/17549


-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org