pjmore opened a new pull request #1071:
URL: https://github.com/apache/arrow-datafusion/pull/1071
# Which issue does this PR close?
Closes #1069.
# Rationale for this change
Make volatility a first class concept for functions.
# What changes are included in this PR?
Renamed the Signature enum to TypeSignature and added Signature struct which
currently has 2 fields, type_signature and volatility. I initially tried adding
another field containing the volatility to the original Signature enum but this
felt clunky and while it could potential allow for conditionally
stable\immutable functions with the OneOf variant that seemed like a fairly
niche optimization.
# Are there any user-facing changes?
Yes Signature was renamed and volatility has been added. There are breaking
changes to the public API from this, could someone add the `api-break` label? I
don't have permission to add labels.
--
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]
