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

Query property graph on unspecified property should return error #190

Open
2 tasks done
dentiny opened this issue Dec 25, 2024 · 0 comments
Open
2 tasks done

Query property graph on unspecified property should return error #190

dentiny opened this issue Dec 25, 2024 · 0 comments

Comments

@dentiny
Copy link
Contributor

dentiny commented Dec 25, 2024

What happens?

As of now, querying unspecified property from graph does not necessarily return error.

For more details, see reproduction code below and comments in thread #26 (comment)

To Reproduce

D CREATE TABLE Person(
      id BIGINT PRIMARY KEY, 
      name VARCHAR
  );
D INSERT INTO Person VALUES (0, 'Foo'), (1, 'Bar');
D -CREATE PROPERTY GRAPH g VERTEX TABLES (Person PROPERTIES (id));
┌─────────┐
│ Success │
│ boolean │
├─────────┤
│ 0 rows  │
└─────────┘
D -SELECT * FROM GRAPH_TABLE (g MATCH (p:Person) COLUMNS (p.id, p.name));
┌───────┬─────────┐
│  id   │  name   │
│ int64 │ varchar │
├───────┼─────────┤
│     0 │ Foo     │
│     1 │ Bar     │
└───────┴─────────┘

This should throw an error message stating, property "NAME" is never defined. (This is what Oracle does, at least.)

@Dtenwolde I copy your comments in the thread to this new issue, just for easier and more clear reference for my PR, hope you don't mind :)

OS:

ubuntu 22.04, x86_64

DuckDB Version:

c380346433

DuckDB Client:

C++

Full Name:

Hao

Affiliation:

N/A

How did you load the extension?

Built from source

Did you include all relevant data sets for reproducing the issue?

Yes

Did you include all code required to reproduce the issue?

  • Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Python version, Linux distribution) to reproduce the issue?

  • Yes, I have
@dentiny dentiny changed the title Query on unspecified property should return error Query property graph on unspecified property should return error Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant