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

Foreign key support #1192

Merged
merged 78 commits into from Dec 7, 2015

Conversation

Projects
None yet
3 participants
@cpcloud
Member

cpcloud commented Aug 2, 2015

related: blaze/odo#274, blaze/datashape#164

Both of the above PRs need to be merged before merging this one

@cpcloud cpcloud added this to the 0.8.3 milestone Aug 2, 2015

@cpcloud cpcloud self-assigned this Aug 2, 2015

@llllllllll

This comment has been minimized.

Member

llllllllll commented Aug 5, 2015

This looks pretty straight forward, I will try to run this locally tomorrow.

@cpcloud cpcloud force-pushed the cpcloud:foreign-keys branch from 021bc61 to cba3f37 Aug 7, 2015

@jreback

This comment has been minimized.

Contributor

jreback commented Nov 23, 2015

@cpcloud this can be merged into master now?

@cpcloud cpcloud force-pushed the cpcloud:foreign-keys branch from 705ab32 to 5355d0a Nov 24, 2015

@cpcloud cpcloud force-pushed the cpcloud:foreign-keys branch from 5355d0a to b014952 Nov 26, 2015

@cpcloud cpcloud modified the milestone: 0.9.0 Dec 4, 2015

@cpcloud

This comment has been minimized.

Member

cpcloud commented Dec 7, 2015

@llllllllll is there anyway you can run some of the quantopian test suite against this PR before we merge it?

@llllllllll

This comment has been minimized.

Member

llllllllll commented Dec 7, 2015

I will run our tests now, thanks

@llllllllll

This comment has been minimized.

Member

llllllllll commented Dec 7, 2015

All tests for our main blaze based project are passing, thanks!

@cpcloud

This comment has been minimized.

Member

cpcloud commented Dec 7, 2015

@llllllllll awesome, thanks for testing!

@cpcloud cpcloud removed the wip label Dec 7, 2015

@cpcloud

This comment has been minimized.

Member

cpcloud commented Dec 7, 2015

ok merging on next pass

cpcloud added a commit that referenced this pull request Dec 7, 2015

@cpcloud cpcloud merged commit 0cc7894 into blaze:master Dec 7, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@cpcloud cpcloud deleted the cpcloud:foreign-keys branch Dec 7, 2015

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