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] Removes transform performance note #55177
Conversation
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/ml-core (:ml/Transform) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Still LGTM. FWIW, future outlook: Once #54862 is available (probably 7.8) we can hint towards throttling in order to limit resource usage. |
aggregation that it performs and the indexing process. | ||
{transforms-cap} perform search aggregations on the source indices then index | ||
the results into the destination index. Therefore, a {transform} never takes | ||
less time than the cumulated duration of the aggregation and the indexing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
never takes less time
This is not just about duration but also about memory and CPU. Perhaps this could say that it never takes less time or uses less resources ..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks!
Related to #54254
This PR removes an out-dated note from the transform performance documentation (https://www.elastic.co/guide/en/elasticsearch/reference/master/transform-overview.html#transform-performance).
Preview: http://elasticsearch_55177.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/transform-overview.html