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.

Sign up for GitHub

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

Jump to bottom

BUG: Memoization should include expression name where available #1080

Closed
wants to merge 8 commits into from
Closed

BUG: Memoization should include expression name where available #1080

wants to merge 8 commits into from

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Jul 26, 2017 •
edited
Loading

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 fix-named-impala-exprs branch 2 times, most recently from 7f10dad to 1a6a90d Compare July 27, 2017 02:12
@cpcloud cpcloud self-assigned this Jul 27, 2017
@cpcloud cpcloud added this to the 0.11.3 milestone Jul 27, 2017
@cpcloud cpcloud added bug Incorrect behavior inside of ibis impala The Apache Impala backend labels Jul 27, 2017
@cpcloud
Copy link
Member Author

cpcloud commented Jul 27, 2017

merging so that we can get builds back to green

@wesm
Copy link
Member

wesm commented Jul 27, 2017

👍

@cpcloud cpcloud closed this in 96acf67 Jul 27, 2017
@cpcloud cpcloud deleted the fix-named-impala-exprs branch July 27, 2017 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cpcloud cpcloud

Labels
bug Incorrect behavior inside of ibis impala The Apache Impala backend
Projects
None yet
Milestone
0.12
Development

Successfully merging this pull request may close these issues.
2 participants
@cpcloud @wesm