Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Clean associative fix #80

Merged
2 commits merged into from Mar 13, 2011

Conversation

Projects
None yet
3 participants
Contributor

l0st3d commented Mar 8, 2011

As per Lau's request, I've rebased the associative-fix branch that I requested was pulled earlier today.

Thanks

doooks and others added some commits Jan 28, 2011

fixed joins not carrying info through to other joins
joins should be sorted by dependency on previously joined tables

e.g. this

SELECT * FROM t1
JOIN t3 ON (t3.c = t2.c)
JOIN t2 ON (t1.a = t2.a)

is not valid, since t3 cannot be joined to anything on t1, without t2.  Should be

SELECT * FROM t1
JOIN t2 ON (t1.a = t2.a)
JOIN t3 ON (t3.c = t2.c)
Owner

LauJensen commented Mar 13, 2011

Thanks a lot - Excellent work!

Now merged to master and released as gem.

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment