Skip to content

fix: cypher pushdown to sql was broken with respect to property names#271

Merged
adsharma merged 2 commits intomasterfrom
duckdb_fix
Mar 5, 2026
Merged

fix: cypher pushdown to sql was broken with respect to property names#271
adsharma merged 2 commits intomasterfrom
duckdb_fix

Conversation

@adsharma
Copy link
Contributor

@adsharma adsharma commented Mar 5, 2026

install duckdb; load duckdb; 
ATTACH 'demo.duckdb' AS demo (dbtype duckdb); 
CREATE REL TABLE lives_in2 (FROM demo.user TO demo.city) WITH (storage = 'demo.lives_in');

and then:

lbug> match (a:demo.user)-[b:lives_in2]->(c:demo.city) return a.*, b.*, c.*;
┌───────┬─────────┬───────┬───────────┬───────────┬───────┬───────────┬──────────────┐
│ a.id  │ a.name  │ a.age │ b.user_id │ b.city_id │ c.id  │ c.name    │ c.population │
│ INT64 │ STRING  │ INT32 │ INT64     │ INT64     │ INT64 │ STRING    │ INT32        │
├───────┼─────────┼───────┼───────────┼───────────┼───────┼───────────┼──────────────┤
│ 2     │ Karissa │ 40    │ 2         │ 1         │ 1     │ Waterloo  │ 150000       │
│ 3     │ Zhang   │ 50    │ 3         │ 2         │ 2     │ Kitchener │ 200000       │
│ 4     │ Noura   │ 25    │ 4         │ 3         │ 3     │ Guelph    │ 75000        │
│ 1     │ Adam    │ 30    │ 1         │ 1         │ 1     │ Waterloo  │ 150000       │
└───────┴─────────┴───────┴───────────┴───────────┴───────┴───────────┴──────────────┘

This was broken earlier.

@adsharma adsharma merged commit b54e480 into master Mar 5, 2026
32 of 33 checks passed
@adsharma adsharma deleted the duckdb_fix branch March 5, 2026 20:47
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