Skip to content

Conversation

coderfender
Copy link
Contributor

Which issue does this PR close?

Documentation update to #2073 changes

Closes #.

Rationale for this change

Doc update

What changes are included in this PR?

Doc update

How are these changes tested?

Local testing

@coderfender
Copy link
Contributor Author

@andygrove , Documentation update to let users know that we now support Try arithmetic

@coderfender
Copy link
Contributor Author

Please review and would love to add / update more docs if needed .

@andygrove
Copy link
Member

Thanks @coderfender could you also update docs/source/user-guide/expressions.md

@coderfender
Copy link
Contributor Author

Thank you for the feedback @andygrove , I updated the expressions doc to provide further info re Try arithmetic functions

Copy link
Contributor

@kazuyukitanimura kazuyukitanimura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving
we have more tests coming in #2136

Copy link
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @coderfender

@andygrove andygrove merged commit 67a031d into apache:main Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants