akashjainn commented on issue #17498: URL: https://github.com/apache/datafusion/issues/17498#issuecomment-5782140515
I'd like to take `datafusion.execution.parquet.compression`. On today's main it is still a plain string, so `SET datafusion.execution.parquet.compression = 'zstdd(3)'` succeeds and the bad value only fails when a file is written. The plan is the same shape as #24642 for `statistics_enabled`: a small typed value that parses the codec and level up front, with `parse_compression_string` kept as a thin wrapper over it so there is one parser. I have it working locally with the level ranges checked against the parquet crate. Will have a PR up soon. -- 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]
