dwsmith1983 commented on issue #7559: URL: https://github.com/apache/datafusion-comet/issues/7559#issuecomment-6756663538
> We should in general redact urls Agreed. #7127 added shared helpers in `url_redaction.rs`, and the #7128 pull requests route the object store, Parquet writer and native Delta paths through them. Iceberg's paths are the ones left, and the fix here would use the same helpers. Once #7561 and #7562 merge, I'd also move their display functions into `url_redaction.rs`, so every native error formats a path the same way. -- 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]
