apache / apache/iotdb

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

オープン
#14,290 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
6.4k
フォーク
1.2k
平均マージ
1日 23時間
マージ済み PR(30日)
115

説明

### Search before asking

- [X] I searched in the [issues](https://github.com/apache/iotdb/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?

- [x] I'm willing to submit a PR!

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Payload には実装ファイルもテストも記載されていません。まず、Issue の最小再現を IoTDB に対して実行し、query 1 と query 2 を比較してください。両方の式が期待される同値の結果を生成し、単項否定が乗算より優先されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java, sql
領域
databases
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。