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

MINOR: Remove Dead Code in SearchScript (#33569) #33594

Merged
merged 4 commits into from
Sep 13, 2018

Conversation

original-brownbear
Copy link
Member

  • lookup is not used anywhere
  • getLeafContext is not used anywhere

backport PR just to run Jenkins

* `lookup` is not used anywhere
* `getLeafContext` is not used anywhere
@original-brownbear original-brownbear added >non-issue :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache labels Sep 11, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@original-brownbear
Copy link
Member Author

Jenkins test this

@original-brownbear
Copy link
Member Author

Jenkins test this

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@original-brownbear original-brownbear merged commit 3414c3a into elastic:6.x Sep 13, 2018
@original-brownbear original-brownbear deleted the 33569-6.x branch September 13, 2018 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >non-issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants