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

[Bug] There is a logical error in the precedence of arithmetic operators. #14290

Open
2 tasks done
LingweiKuang opened this issue Dec 3, 2024 · 1 comment
Open
2 tasks done

Comments

@LingweiKuang
Copy link

LingweiKuang commented Dec 3, 2024

Search before asking

  • I searched in the issues and found nothing similar.

Version

version 1.3.3 (Build: ad95a7e)

Describe the bug and provide the minimal reproduce step

DROP DATABASE root.db0
CREATE DATABASE root.db0

CREATE TIMESERIES root.db0.t1 WITH datatype=INT32;
INSERT INTO root.db0(timestamp, t1) VALUES (1641024000000, 0);

# query 1
SELECT (t1 * -1000) FROM root.db0

# query 2
SELECT (t1 * (-1000)) FROM root.db0

What did you expect to see?

Query 1 returned result set: -0

Query 2 returned result set: -0

What did you see instead?

Query 1 returned result set: -1000

Query 2 returned result set: -0

Anything else?

Dear IoTDB team, In the above scenario, we combine the unary negation operator with column t1 and perform a multiplication operation to examine the result returned by IotDB. In Query 1, we did not enclose -t1 in parentheses to treat it as a whole, and the returned result was -1000, which does not match our expectations. In principle, the unary negation operator should have a higher precedence than the multiplication operator, and the presence or absence of parentheses should not affect the final result.

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@LingweiKuang
Copy link
Author

There is a logical error in the precedence and association of arithmetic operators

@LingweiKuang LingweiKuang changed the title [Bug] There is a logical error in the precedence and association of arithmetic operators. [Bug] There is a logical error in the precedence of arithmetic operators. Dec 3, 2024
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

No branches or pull requests

1 participant