kosiew commented on code in PR #14887: URL: https://github.com/apache/datafusion/pull/14887#discussion_r1970909759
########## datafusion/sql/src/planner.rs: ########## @@ -43,15 +43,30 @@ pub use datafusion_expr::planner::ContextProvider; /// SQL parser options #[derive(Debug, Clone, Copy)] pub struct ParserOptions { + /// Whether to parse float as decimal. pub parse_float_as_decimal: bool, + /// Whether to normalize identifiers. pub enable_ident_normalization: bool, + /// Whether to support varchar with length. pub support_varchar_with_length: bool, + /// Whether to normalize options value. pub enable_options_value_normalization: bool, + /// Whether to collect spans pub collect_spans: bool, Review Comment: I did not add getter functions for the fields because they are `pub` and did not want to make API change by making them private. -- 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