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

[Painless] Add instance bindings #34410

Open
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@jdconrad
Contributor

jdconrad commented Oct 11, 2018

This change adds instance bindings to Painless. This binding allows a whitelisted method to be called on an instance instantiated prior to script compilation. Whitelisting must be done in code as there is no practical way to instantiate a useful instance from a text file (see the tests for an example). Since an instance can be shared by multiple scripts, each method called must be thread-safe.

@elasticmachine

This comment has been minimized.

Show comment
Hide comment
@elasticmachine

elasticmachine commented Oct 11, 2018

@jdconrad jdconrad referenced this pull request Oct 16, 2018

Open

Painless: Roadmap #31009

2 of 20 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment