Skip to content

feat: degrees and radians#6023

Merged
yjshen merged 4 commits intoapache:mainfrom
izveigor:degrees_radians
Apr 19, 2023
Merged

feat: degrees and radians#6023
yjshen merged 4 commits intoapache:mainfrom
izveigor:degrees_radians

Conversation

@izveigor
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Part of #5838

Rationale for this change

What changes are included in this PR?

Implementation "Degrees" and "Radians" SQL functions.

Are these changes tested?

Yes

Are there any user-facing changes?

Yes

@github-actions github-actions bot added core Core DataFusion crate logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt) labels Apr 15, 2023
Copy link
Copy Markdown
Member

@waynexia waynexia left a comment

Choose a reason for hiding this comment

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

Looking good. There are some conflicts

@yjshen yjshen merged commit 427231e into apache:main Apr 19, 2023
@izveigor izveigor mentioned this pull request Apr 19, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core DataFusion crate logical-expr Logical plan and expressions physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants