andygrove commented on issue #631: URL: https://github.com/apache/datafusion-comet/issues/631#issuecomment-2228948852
> @andygrove QQ: Upon checking i found out that DataFusion doesn't currently support a built-in `to_json` function. While implementing it directly in Comet is an option, there might be a more efficient approach by implementing the function in datafusion and reusing it here. What are your thoughts on these considerations? @dharanad It makes sense to add this in DataFusion. I see that Postgres also supports `to_json`. Spark supports a number of options in `to_json` to control how dates and times are formatted. As long as those options are also available in the DataFusion version then I think we should be able to reuse it directly. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
