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

BUG: Memoization should include expression name where available #1080

Closed
wants to merge 8 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@cpcloud
Member

cpcloud commented Jul 26, 2017

Also:

  • fix docker builds
  • upgrade docker image to cpcloud86/impala:java8
  • remove most bare except:s
  • fix a broken doc test

@cpcloud cpcloud force-pushed the cpcloud:fix-named-impala-exprs branch 2 times, most recently from 7f10dad to 1a6a90d Jul 27, 2017

@cpcloud cpcloud self-assigned this Jul 27, 2017

@cpcloud cpcloud added this to the 0.11.3 milestone Jul 27, 2017

@cpcloud cpcloud force-pushed the cpcloud:fix-named-impala-exprs branch from 1a6a90d to 09e4632 Jul 27, 2017

@cpcloud cpcloud force-pushed the cpcloud:fix-named-impala-exprs branch from 09e4632 to 8f8d5c2 Jul 27, 2017

@cpcloud cpcloud force-pushed the cpcloud:fix-named-impala-exprs branch from ca2674c to f8cc86b Jul 27, 2017

cpcloud added some commits Jul 27, 2017

@cpcloud

This comment has been minimized.

Member

cpcloud commented Jul 27, 2017

merging so that we can get builds back to green

@wesm

This comment has been minimized.

Member

wesm commented Jul 27, 2017

👍

@cpcloud cpcloud closed this in 96acf67 Jul 27, 2017

@cpcloud cpcloud deleted the cpcloud:fix-named-impala-exprs branch Jul 27, 2017

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