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

56 make columns clause optional #194

Merged
merged 12 commits into from
Mar 4, 2024
Merged

Conversation

Dtenwolde
Copy link
Collaborator

Similar to DuckDB's parser where the SELECT is optional, omitting it implies a SELECT *, the COLUMNS clause is now also optional. This creates a COLUMNS(*) which returns all the columns, it also allows you to not repeat the columns explicitly in case you want to use them in the outer SELECT clause i.c.w. an aggregate or additional filter.

Sidenote: Also the trailing comma no longer leads to a parser error. COLUMNS(p.id,) is allowed for instance.

The simplest example:

-FROM GRAPH_TABLE (snb MATCH (p:Person)) limit 1

Equal to

FROM person
limit 1

@Dtenwolde Dtenwolde merged commit e55aa6d into main Mar 4, 2024
32 of 45 checks passed
@Dtenwolde Dtenwolde deleted the 56-make-columns-clause-optional branch March 4, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant