Skip to content

feat(spark): Adds spark round function#21062

Merged
alamb merged 7 commits intoapache:mainfrom
SubhamSinghal:add-spark-round-function
Apr 3, 2026
Merged

feat(spark): Adds spark round function#21062
alamb merged 7 commits intoapache:mainfrom
SubhamSinghal:add-spark-round-function

Conversation

@SubhamSinghal
Copy link
Copy Markdown
Contributor

What changes are included in this PR?

Adds support for round spark function in data fusion.

Are these changes tested?

yes, using UTs

Are there any user-facing changes?

yes, adds new function.

@github-actions github-actions Bot added sqllogictest SQL Logic Tests (.slt) spark labels Mar 19, 2026
Comment thread datafusion/spark/src/function/math/round.rs Outdated
Comment thread datafusion/spark/src/function/math/round.rs Outdated
@SubhamSinghal SubhamSinghal requested a review from martin-g March 20, 2026 13:06
Comment thread datafusion/spark/Cargo.toml Outdated
@SubhamSinghal
Copy link
Copy Markdown
Contributor Author

@martin-g @alamb fixed build failure. it can be merged now.

@alamb alamb added this pull request to the merge queue Apr 3, 2026
@alamb
Copy link
Copy Markdown
Contributor

alamb commented Apr 3, 2026

Thanks @SubhamSinghal and @martin-g

Merged via the queue into apache:main with commit d143bfa Apr 3, 2026
34 checks passed
@SubhamSinghal SubhamSinghal deleted the add-spark-round-function branch April 4, 2026 03:30
Rich-T-kid pushed a commit to Rich-T-kid/datafusion that referenced this pull request Apr 21, 2026
## What changes are included in this PR?

Adds support for round spark function in data fusion.

## Are these changes tested?

yes, using UTs

## Are there any user-facing changes?

yes, adds new function.

---------

Co-authored-by: Subham Singhal <subhamsinghal@Subhams-MacBook-Air.local>
Co-authored-by: Martin Grigorov <martin-g@users.noreply.github.com>
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

spark sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants