Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Nan related #643

Closed
czpmango opened this issue Jan 20, 2021 · 0 comments
Closed

Nan related #643

czpmango opened this issue Jan 20, 2021 · 0 comments
Labels
incompatible PR: incompatible with the master branches in the storage or common repos need to discuss PR: need to discuss the current PR

Comments

@czpmango
Copy link
Contributor

czpmango commented Jan 20, 2021

Nebula does not define NaN and some computational behavior is inconsistent with Neo4j.
But I think Nebula's test result is also reasonable.If confirmed, I will close this issue.

neo4j test:

neo4j@neo4j> return 0.0/0.0;
+---------+
| 0.0/0.0 |
+---------+
| NaN     |
+---------+

1 row available after 24 ms, consumed after another 0 ms
neo4j@neo4j> return 0.0/0.0>'a';
+-------------+
| 0.0/0.0>'a' |
+-------------+
| NULL        |
+-------------+

nebula test:

(czp@nebula) [nba]> yield 0.0/0.0
+-------------+
| (0/0)       |
+-------------+
| DIV_BY_ZERO |
+-------------+
Got 1 rows (time spent 1222/1749 us)

Wed, 20 Jan 2021 10:08:28 CST

(czp@nebula) [nba]> yield 0.0/0.0>'a'
+-------------+
| ((0/0)>a)   |
+-------------+
| DIV_BY_ZERO |
+-------------+
@czpmango czpmango added the need to discuss PR: need to discuss the current PR label Jan 20, 2021
@whitewum whitewum added the incompatible PR: incompatible with the master branches in the storage or common repos label Jan 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
incompatible PR: incompatible with the master branches in the storage or common repos need to discuss PR: need to discuss the current PR
Projects
None yet
Development

No branches or pull requests

3 participants