cayleygraph / cayley

An open-source graph database
https://cayley.io
Apache License 2.0
14.86k stars 1.25k forks source link

LinkedQL #878

Closed iddan closed 4 years ago

iddan commented 5 years ago

Resolves #877 Resolves #835 Resolves #852 Resolves #311

This change is Reviewable

iddan commented 5 years ago

Status of implementation:

iddan commented 5 years ago

@dennwc I'd like your help with Morphism. I'll try to reduce panics and cover more parts of the code with testing soon but I think we can start the review process.

iddan commented 5 years ago

I see what you say. But I think the steps should reflect the API exposed. Specifically, I think .skip().limit() is a good API. I think two guidelines are similarity to Gremlin API and simplicity.

iddan commented 5 years ago

@dennwc I think I got rid of the last problematic panic()

iddan commented 4 years ago

@dennwc I think for now we can leave Properties to accept only IRIs until we make changes for morphism.

iddan commented 4 years ago

@dennwc we should make properties with empty iri set to save all properties