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

Delete tag from multiple vertices produces Invalid result #5262

Closed
Aiee opened this issue Jan 16, 2023 · 3 comments
Closed

Delete tag from multiple vertices produces Invalid result #5262

Aiee opened this issue Jan 16, 2023 · 3 comments
Labels
affects/none PR/issue: this bug affects none version. process/fixed Process of bug severity/none Severity of bug type/bug Type: something is unexpected wontfix Solution: this will not be worked on recently

Comments

@Aiee
Copy link
Contributor

Aiee commented Jan 16, 2023

Please check the FAQ documentation before raising an issue

Describe the bug (required)

Your Environments (required)

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

How To Reproduce(required)

Steps to reproduce the behavior:

(root@nebula) [nba]>       FETCH PROP ON player "Tim Duncan" YIELD player.name, player.age

+--------------+------------+
| player.name  | player.age |
+--------------+------------+
| "Tim Duncan" | 42         |
+--------------+------------+
Got 1 rows (time spent 9.768ms/10.481071ms)

Mon, 16 Jan 2023 17:01:48 CST

(root@nebula) [nba]>       FETCH PROP ON player "Tony Parker" YIELD player.name, player.age
+---------------+------------+
| player.name   | player.age |
+---------------+------------+
| "Tony Parker" | 36         |
+---------------+------------+
Got 1 rows (time spent 3.767ms/4.299134ms)

Mon, 16 Jan 2023 17:02:24 CST

(root@nebula) [nba]> 
(root@nebula) [nba]>       DELETE TAG player FROM "Tim Duncan", "Tony Parker";
Execution succeeded (time spent 5.867ms/6.275803ms)

Mon, 16 Jan 2023 17:21:38 CST

(root@nebula) [nba]>       FETCH PROP ON player "Tim Duncan" YIELD player.name, player.age

+-------------+------------+
| player.name | player.age |
+-------------+------------+
|             |            |
+-------------+------------+
Got 1 rows (time spent 4.478ms/4.93895ms)

Mon, 16 Jan 2023 17:21:41 CST

(root@nebula) [nba]>       FETCH PROP ON player "Tony Parker" YIELD player.name, player.age
+-------------+------------+
| player.name | player.age |
+-------------+------------+
+-------------+------------+
Empty set (time spent 4.355ms/4.910096ms)

Expected behavior

FETCH PROP ON player "Tim Duncan" YIELD player.name, player.age should return an empty set

Additional context

@Aiee Aiee added the type/bug Type: something is unexpected label Jan 16, 2023
@github-actions github-actions bot added affects/none PR/issue: this bug affects none version. severity/none Severity of bug labels Jan 16, 2023
@czpmango
Copy link
Contributor

czpmango commented Jan 17, 2023

IMHO, returning one empty row is somewhat expected.
But, we should redefine the behavior of the RPC interface getProps and make it clearer.

@Aiee Aiee mentioned this issue Jan 17, 2023
11 tasks
@Aiee
Copy link
Contributor Author

Aiee commented Jan 18, 2023

This is by design, Tim Duncan has multiple tags, so we return the empty row to indicate the vertex still exists, while Tony Parker has only one tag named player.

@Aiee Aiee added the wontfix Solution: this will not be worked on recently label Jan 19, 2023
@xtcyclist
Copy link
Contributor

Close this issue as it is by design and won't fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/none PR/issue: this bug affects none version. process/fixed Process of bug severity/none Severity of bug type/bug Type: something is unexpected wontfix Solution: this will not be worked on recently
Projects
None yet
Development

No branches or pull requests

3 participants