Skip to content

docs(querying): correct ceil, floor, and round return type to float - #1861

Merged
katarinasupe merged 4 commits into
memgraph:mainfrom
siddiqueirshad:fix/functions-return-type-float
Oct 28, 2026
Merged

katarinasupe merged 4 commits into
memgraph:mainfrom
siddiqueirshad:fix/functions-return-type-float

Conversation

@siddiqueirshad

Copy link
Copy Markdown
Contributor

Summary

According to the openCypher specification and Memgraph runtime evaluation, \ceil(), \ loor(), and
ound()\ return floating-point numbers (\FLOAT) rather than integers.

In \pages/querying/functions.mdx, their signature tables previously listed -> (integer). This PR updates the signature definitions to accurately reflect -> (float)\ across all three functions.

Fixes memgraph/memgraph#5201

@vercel

vercel Bot commented Oct 26, 2026

Copy link
Copy Markdown

@siddiqueirshad is attempting to deploy a commit to the Memgraph Team on Vercel.

A member of the Team first needs to authorize it.

@katarinasupe katarinasupe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for catching this and for the clear fix! I checked the source, and ceil, floor and round do return FLOAT.

One more thing while you're in this table: all three also accept integers, so the input type is wrong too. Could you change the signatures to:

ceil(number: integer|float) -> (float)
floor(number: integer|float) -> (float)
round(number: integer|float) -> (float)

That matches how exp, log and the others are listed. Once that's in, I'll approve.

@siddiqueirshad

Copy link
Copy Markdown
Contributor Author

Hi @katarinasupe, thanks for the review! I have updated the input signatures for ceil, loor, and
ound to integer|float as requested. Please let me know if anything else is needed.

@katarinasupe

Copy link
Copy Markdown
Contributor

Thank you @siddiqueirshad! I will merge this to main once the checks are done.
What are you using Memgraph for? I would love to hear about it - do you have a project to share?

@vercel

vercel Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
documentation Ready Ready Preview Oct 28, 2026 10:23am UTC

Request Review

@katarinasupe
katarinasupe merged commit 9be0929 into memgraph:main Oct 28, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
Preview — dbaf27d7 Deployed Oct 28, 2026 by vercel[bot]
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.

ceil(), floor(), and round() return FLOAT despite documented INTEGER signatures

2 participants