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

Join names #1208

Merged
merged 3 commits into from Aug 18, 2015

Conversation

Projects
None yet
2 participants
@llllllllll
Member

llllllllll commented Aug 17, 2015

sqlalchemy was raising warnings and you couldn't actually access the left and right sides any more.

@llllllllll llllllllll added this to the 0.8.3 milestone Aug 17, 2015

@llllllllll

This comment has been minimized.

Member

llllllllll commented Aug 17, 2015

looks like dask is complaining about some stuff, did anything change here recently?

@cpcloud

This comment has been minimized.

Member

cpcloud commented Aug 17, 2015

dask is changing fairly rapidly these days

ping @mrocklin

@cpcloud

This comment has been minimized.

Member

cpcloud commented Aug 17, 2015

ok this dask stuff is blocking, taking care of it now

@cpcloud

This comment has been minimized.

Member

cpcloud commented Aug 17, 2015

very small issue .... going to push directly to master

@cpcloud

This comment has been minimized.

Member

cpcloud commented Aug 17, 2015

fixed in 376d87b

@cpcloud

This comment has been minimized.

Member

cpcloud commented Aug 17, 2015

sorry ... your whatsnew conflicts with the one i just merged in :|

@llllllllll llllllllll force-pushed the quantopian:join-names branch from ef338d9 to 8989548 Aug 17, 2015

@llllllllll

This comment has been minimized.

Member

llllllllll commented Aug 17, 2015

rebased, does the change look correct, if so I will merge on pass.

llllllllll added a commit that referenced this pull request Aug 18, 2015

@llllllllll llllllllll merged commit e518f4e into blaze:master Aug 18, 2015

1 check passed

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

@llllllllll llllllllll deleted the quantopian:join-names branch Aug 18, 2015

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