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

added support for multiple joins #8

Closed
wants to merge 7 commits into from
Closed

added support for multiple joins #8

wants to merge 7 commits into from

Conversation

brunoguadagnini
Copy link

No description provided.

@darrachequesne
Copy link
Owner

Hi! If not explicitly set, it seems that an INNER JOIN clause is created with path.get() instead of a LEFT JOIN, thus the failure.

Related (I guess): #7

@brunoguadagnini
Copy link
Author

Hi @darrachequesne!
Yes, it is the same reported #7! Sorry, I did not see!

@darrachequesne
Copy link
Owner

hi @brunoguadagnini, I can't remember exactly why that didn't work back then, but multiple joins should now properly work 👼

Added in 5ba97e9

@brunoguadagnini
Copy link
Author

Great job @darrachequesne!
Thank you very much!

@vbura
Copy link

vbura commented Dec 12, 2016

Thank you very much!
Good job:)

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.

3 participants