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

Allow painless to load stored fields #24290

Merged
merged 1 commit into from Apr 24, 2017

Conversation

@nik9000
Copy link
Contributor

commented Apr 24, 2017

We document that painless can load stored fields but it can't
because the classes that make that work aren't whitelisted.

Allow painless to load stored fields
We document that painless can load stored fields but it can't
because the classes that make that work aren't whitelisted.
@nik9000

This comment has been minimized.

Copy link
Contributor Author

commented Apr 24, 2017

I found this while running down a CI failure. The doc tests were passing because we didn't refresh. When I added refreshes the tests started falling over.

@jdconrad
Copy link
Contributor

left a comment

Good catch. Thanks for fixing this.

@nik9000 nik9000 added v5.5.0 v6.0.0-alpha1 and removed discuss labels Apr 24, 2017

@nik9000 nik9000 merged commit 5fbc86e into elastic:master Apr 24, 2017

2 checks passed

CLA Commit author has signed the CLA
Details
elasticsearch-ci Build finished.
Details

@nik9000 nik9000 added the v5.4.0 label Apr 24, 2017

nik9000 added a commit that referenced this pull request Apr 24, 2017
Allow painless to load stored fields (#24290)
We document that painless can load stored fields but it can't
because the classes that make that work aren't whitelisted.
nik9000 added a commit that referenced this pull request Apr 24, 2017
Allow painless to load stored fields (#24290)
We document that painless can load stored fields but it can't
because the classes that make that work aren't whitelisted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.