akashjainn commented on issue #17498: URL: https://github.com/apache/datafusion/issues/17498#issuecomment-5782424128
I'd like to take `datafusion.execution.parquet.coerce_int96` next. On today's main it is still a plain string: `SET datafusion.execution.parquet.coerce_int96 = 'nanos'` succeeds, `SHOW` reports it, and the error only comes on the first Parquet scan. #20253 did this in January and got a full review, but it went stale and was closed in June. I have a version working locally that follows that review: a small `DFTimeUnit` in `parquet_config.rs`, the docstring that reads as a boolean rewritten, and the parser in `source.rs` with its `unwrap()` removed since a typed field has nothing left to parse. Will have PR up shortly. -- 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]
