Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Floor & ceiling functions give wrong results with exact numeric arguments [CORE1677] #2102

Closed
firebird-issue-importer opened this issue Jan 7, 2008 · 12 comments

Comments

@firebird-issue-importer
Copy link

@firebird-issue-importer firebird-issue-importer commented Jan 7, 2008

Submitted by: stephano (stephano)

Is related to QA207

select floor(cast(1500 as numeric(18,5))) from rdb$database -> -4827 (wrong)
select floor(cast(1500 as numeric(18,4))) from rdb$database -> 1500 (correct)
select ceiling(cast(1500 as numeric(18,5))) from rdb$database -> -4826 (wrong)

Actually, any precision higher than 6 gives a wrong result.

Commits: 1c40b7d

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 7, 2008

Modified by: stephano (stephano)

description: select floor(cast(1500 as numeric(18,5))) from rdb$database -> -4827 (wrong)
select floor(cast(1500 as numeric(18,4))) from rdb$database -> 1500 (correct)

Actually, any precision higher than 6 gives a wrong result.

=>

select floor(cast(1500 as numeric(18,5))) from rdb$database -> -4827 (wrong)
select floor(cast(1500 as numeric(18,4))) from rdb$database -> 1500 (correct)
select ceiling(cast(1500 as numeric(18,5))) from rdb$database -> -4826 (wrong)

Actually, any precision higher than 6 gives a wrong result.

summary: Floor functions gives wrong result with exact numeric argument => Floor & ceiling functions give wrong results with exact numeric arguments

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 7, 2008

Modified by: @asfernandes

assignee: Adriano dos Santos Fernandes [ asfernandes ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 7, 2008

Commented by: @asfernandes

The bug is fixed... I'm maintaining the ticket opened because:
1) I don't know if it can be included in RC1
2) There is no 2.1 RC2 version in the tracker

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 8, 2008

Modified by: @dyemanov

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 2.1 RC1 [ 10201 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 28, 2008

Modified by: @pcisar

Workflow: jira [ 13802 ] => Firebird [ 14114 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 13, 2008

Modified by: @pmakowski

Link: This issue block progress on QA207 [ QA207 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 13, 2008

Commented by: @pmakowski

Q/A test made ok

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Sep 13, 2008

Modified by: @pmakowski

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 26, 2009

Modified by: @pcisar

Link: This issue is related to QA207 [ QA207 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 26, 2009

Modified by: @pcisar

Link: This issue block progress on QA207 [ QA207 ] =>

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 19, 2016

Modified by: @pavel-zotov

QA Status: No test

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 19, 2016

Modified by: @pavel-zotov

QA Status: No test => Done successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants