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

Sqlitelogictest - Algebra operators priority in select query #6494

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

Sqlitelogictest - Algebra operators priority in select query #6494

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2017-12-13 15:49:24 +0100
From: @PedroTadim
To: SQL devs <>
Version: -- development

Last updated: 2018-02-12 16:12:18 +0100

Comment 26000

Date: 2017-12-13 15:49:24 +0100
From: @PedroTadim

User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.84 Safari/537.36
Build Identifier:

The algebra operators priority is not followed in this query: SELECT 85 * + 52 * 50 * ( + + 82 ) + - 78; due to lack of parenthesis over +52

Reproducible: Always

Steps to Reproduce:

  1. SELECT 85 * + 52 * 50 * ( + + 82 ) + - 78;

Actual Results:

-62

Expected Results:

18121922

Removing the first plus sign gives the right result:

SELECT 85 * 52 * 50 * ( + + 82 ) + - 78;

Comment 26004

Date: 2017-12-13 16:30:13 +0100
From: MonetDB Mercurial Repository <>

Changeset 6f3ac2786528 made by Pedro Ferreira pedro.ferreira@monetdbsolutions.com in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=6f3ac2786528

Changeset description:

Added tests for bugs 6494, 6495 and 6496

Comment 26013

Date: 2017-12-15 16:38:06 +0100
From: MonetDB Mercurial Repository <>

Changeset b35a861df01d made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=b35a861df01d

Changeset description:

Use "largest" type as result type in atom_{add,sub,mul}.
This fixes bug #6494.
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
1 participant