Omega359 commented on PR #12822: URL: https://github.com/apache/datafusion/pull/12822#issuecomment-2402261823
I find this intriguing and technically very interesting! I do have a few questions: - What are you goals with this? I am assuming cleaner looking/easier to create docs. If so I would wonder how it would help in cases were there is significant amounts of docs vs boilerplate such as in the [to_date](https://github.com/apache/datafusion/blob/3d347c99b32715690453add7fca1d76a3f8bbb01/datafusion/functions/src/datetime/to_date.rs#L84) udf. - Can it be made to handle n arguments? - Is it possible to generate rustdoc with this approach? - I noticed that this approach creates a Documentation object on every call. It's obviously pretty minor but it would be nice if it could be static. To me in general while I find this quite interesting I wonder if the benefit outweighs the added complexity of the code generation. To be clear I am absolutely not against this approach - I just have a few concerns. -- 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]
