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.

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

[ElasticSearch] Fix: wrong variable name. #1521

Merged
merged 1 commit into from
Jan 13, 2017
Merged

[ElasticSearch] Fix: wrong variable name. #1521

merged 1 commit into from
Jan 13, 2017

Conversation

arikfr
Copy link
Member

@arikfr arikfr commented Jan 13, 2017

Fixes #1519.

@adamlwgriffiths FYI.

@arikfr arikfr merged commit 8f1750b into master Jan 13, 2017
@arikfr arikfr deleted the arikfr-patch-1 branch January 13, 2017 20:32
@adamlwgriffiths
Copy link
Contributor

Sorry, I'm not sure how these are getting past my tests =/

arikfr added a commit that referenced this pull request Jan 16, 2017
dairyo pushed a commit to KiiCorp/redash that referenced this pull request Mar 1, 2019
The merge from getredash#1521 fixed the invalid variable error
but it fixed in in a way that introduced another error:
"dictionary changed size during iteration"
This correctly applies the fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants