alamb commented on issue #18319: URL: https://github.com/apache/datafusion/issues/18319#issuecomment-3799533539
> That being said, if the preimage API can be easily extended to other functions then I think preimage is a great name. For example, if a function can register its inverse (its "preimage function" or inverse function, `f^-1`). I tried looking at the API just now and it can probably be expanded beyond interval arithmetic? The two functions we have identified are date_part and date_trunc. - [ ] https://github.com/apache/datafusion/issues/18319 - [ ] https://github.com/apache/datafusion/issues/19889 If we can figure out how to extend the API that would be great. A good usecase would probably help a lot. > So while the terminology is accurate, it does add some cognitive overhead for something that is conceptually quite simple. I do agree with you that the terminology is not commonly used (and I had to look up what pre-image meant, as it seems a somewhat obscure mathematical concept) I made a PR to try and simplify the docs here: - https://github.com/apache/datafusion/pull/20008 -- 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]
