r3stl355 commented on PR #9118: URL: https://github.com/apache/arrow-datafusion/pull/9118#issuecomment-1925413721
Few issues with the docstrings. I can resolve some by using lib names instead of fully qualified (e.g. `datafusion_expr` instead of `datafusion::logical_expr`) but `arrow_schema::DataType` requires an `arrow-schema` dependency which I don't want to add to Cargo.toml just for this. Should I mark the docstring as `ignore`? -- 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]
