alamb commented on issue #8157: URL: https://github.com/apache/arrow-datafusion/issues/8157#issuecomment-1858552912
Update here is that @edmondop tried to implement what is suggested in this issue (❤️ ) and we got a bit hung up on being able to keep the expr_api backwards compatible. See https://github.com/apache/arrow-datafusion/pull/8447#issuecomment-1849027157 I took another shot at trying to implement `expr_fns` such as `encode()` and `decode()` in https://github.com/apache/arrow-datafusion/pull/8046 and I think I got it to work without having to change the public API (it does require moving the expr_fns into the same module as the functions, but I think that is unavoidable if we want to pull functions into their own modules) So therefore I think we might be able to close this issue as "won't fix" -- what do you think @edmondop and @2010YOUY01 ? If we do do that, we should consider what to do with the existing Name function defintion -- 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]
