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

graphd report unexpected error Wrong type result, the type should be NULL, EMPTY, BOOL #5097

Closed
nebula-bots opened this issue Dec 23, 2022 · 1 comment · Fixed by #5102
Closed
Assignees
Labels
affects/master PR/issue: this bug affects master version. auto-sync find/automation process/done Process of bug severity/major Severity of bug type/bug Type: something is unexpected
Milestone

Comments

@nebula-bots
Copy link
Contributor

Please check the FAQ documentation before raising an issue

Describe the bug (required)

Similar to #1574 but that's a fixed issued and we are using the latest master branch.
While executing the following query, graph report an unexpected error:

(root@nebula) [gdlancer]> MATCH p0 = (v0)-[e0]->()<-[e1:Rel_2]-(v1) WHERE (id(v0) in [13, 11, 20, 21, 2, 23, 28, 27, 9, 14, 19]) AND (e0.Rel_0_0_Bool) UNWIND nodes(p0) AS ua0 WITH e0, ua0.Label_0.Label_0_3_Bool AS pa0, ua0, 0.915546 AS pa1 WHERE ua0.Label_0.Label_0_3_Bool RETURN count(*) as totalRows
[ERROR (-1005)]: Wrong type result, the type should be NULL, EMPTY, BOOL

Mon, 12 Dec 2022 18:52:51 CST

(root@nebula) [gdlancer]>

In contrast the same query Neo4j reports no error:

$ MATCH p0 = (v0)-[e0]->()<-[e1:Rel_2]-(v1) WHERE (v0.id in [13, 11, 20, 21, 2, 23, 28, 27, 9, 14, 19]) AND (e0.Rel_0_0_Bool) UNWIND nodes(p0) AS ua0 WITH e0, ua0.Label_0_3_Bool AS pa0, ua0, 0.915546 AS pa1 WHERE ua0.Label_0_3_Bool RETURN count(*) as totalRows

╒═══════════╕
│"totalRows"│
╞═══════════╡
│1          │
└───────────┘

Your Environments (required)

  • OS: uname -a
  • Compiler: g++ --version or clang++ --version
  • CPU: lscpu
  • Commit id (e.g. a3ffc7d8) 573f7f4 (community edition)

How To Reproduce(required)

Steps to reproduce the behavior:

  1. Step 1
  2. Step 2
  3. Step 3

Expected behavior

Additional context

@nebula-bots nebula-bots added affects/master PR/issue: this bug affects master version. auto-sync find/automation severity/major Severity of bug type/bug Type: something is unexpected labels Dec 23, 2022
@xtcyclist xtcyclist self-assigned this Dec 23, 2022
@xtcyclist
Copy link
Contributor

xtcyclist commented Dec 23, 2022

This is related to the usage of NullType::BAD_TYPE, which we have decided to keep the status quo. I've improved the error message to remind users to write boolean_condition == true/false to bypass this. Not going to apply further fixes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/master PR/issue: this bug affects master version. auto-sync find/automation process/done Process of bug severity/major Severity of bug type/bug Type: something is unexpected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants