zhuqi-lucas commented on code in PR #16290: URL: https://github.com/apache/datafusion/pull/16290#discussion_r2139687609
########## datafusion/common/src/config.rs: ########## @@ -264,6 +264,11 @@ config_namespace! { /// Default is false. pub map_varchar_to_utf8view: bool, default = true + /// If true, `CHAR` and `Text` and `String` is mapped to `Utf8View` during SQL planning. + /// If false, `CHAR` and `Text` and `String` is mapped to `Utf8` during SQL planning. + /// Default is true. + pub map_char_to_utf8view: bool, default = true + Review Comment: Thank you @xudong963 for review, this is true, we can use one config, maybe the name should be: map_sql_string_to_utf8view for unifed all sql string to utf8view -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org