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

Add benchmark for painless vs expression scripts #42395

Closed
rjernst opened this issue May 22, 2019 · 3 comments
Closed

Add benchmark for painless vs expression scripts #42395

rjernst opened this issue May 22, 2019 · 3 comments
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache help wanted adoptme Team:Core/Infra Meta label for core/infra team

Comments

@rjernst
Copy link
Member

rjernst commented May 22, 2019

The implementation of expression scripts is difficult to keep in line with changes meant to make painless more performant. Additionally, since expressions are (essentially) a subset of painless, it would be nice to eventually remove them. However, before this can be done, we need to know painless has been brought up to par on performance with expression scripts.

This issue is to add a set of micro benchmarks (and maybe rally tracks as well) that we can monitor and compare the difference in performance between painless and expression scripts.

@rjernst rjernst added help wanted adoptme :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache labels May 22, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@drawlerr drawlerr self-assigned this Jul 12, 2019
@stu-elastic
Copy link
Contributor

Migrated from: #31009

@rjernst rjernst added the Team:Core/Infra Meta label for core/infra team label May 4, 2020
@rjernst rjernst added the needs:triage Requires assignment of a team area label label Dec 3, 2020
@jdconrad jdconrad removed the needs:triage Requires assignment of a team area label label Dec 9, 2020
@rjernst
Copy link
Member Author

rjernst commented May 25, 2024

This has been open for quite a while, and we haven't made much progress on this due to focus in other areas. For now I'm going to close this as something we aren't planning on implementing. We can re-open it later if needed.

@rjernst rjernst closed this as completed May 25, 2024
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 help wanted adoptme Team:Core/Infra Meta label for core/infra team
Projects
None yet
Development

No branches or pull requests

5 participants