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

[DOCS] Clarify stored scripts documentation #119919

Open
consulthys opened this issue Jan 10, 2025 · 2 comments · May be fixed by #119921
Open

[DOCS] Clarify stored scripts documentation #119919

consulthys opened this issue Jan 10, 2025 · 2 comments · May be fixed by #119921
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >docs General docs changes >enhancement Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team

Comments

@consulthys
Copy link
Contributor

In the official documentation about stored scripts (here and but also here), we state that the compilation time is reduced for stored scripts compared to inline scripts, but it turns out that this is not true.

Stored script are also not faster to execute than inline scripts, but the fact that they are stored in the cluster state might reduce the request payload and latency depending on how big the script is and how frequently requests are sent.

We should revise the verbiage around stored scripts to better reflect the above.

@consulthys consulthys added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >docs General docs changes >enhancement Team:Docs Meta label for docs team labels Jan 10, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Jan 10, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@consulthys consulthys changed the title [DOCS] Clarify if/why stored scripts reduce compilation time [DOCS] Clarify stored scripts documentation Jan 10, 2025
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 >docs General docs changes >enhancement Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants