Skip to content

Conversation

@jdconrad
Copy link
Contributor

This adds utility methods to each type of runtime field to return the results of a document in an ordered array based on the same order that doc values are ordered in. This is useful for supporting execute api in this #71374.

@jdconrad jdconrad added >enhancement :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache v8.0.0 v7.13.0 labels Apr 12, 2021
@jdconrad jdconrad requested a review from nik9000 April 12, 2021 22:00
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Apr 12, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me.

@jdconrad
Copy link
Contributor Author

@nik9000 Thanks for the fast review! Will commit once CI passes.

@jdconrad jdconrad merged commit bc11a34 into elastic:master Apr 12, 2021
jdconrad added a commit to jdconrad/elasticsearch that referenced this pull request Apr 13, 2021
…elastic#71599)

This adds utility methods to each type of runtime field to return the results of a document in an ordered array based on the same order that doc values are ordered in. This is useful for supporting execute api in this elastic#71374.
jdconrad added a commit that referenced this pull request Apr 13, 2021
…#71599) (#71604)

This adds utility methods to each type of runtime field to return the results of a document in an ordered array based on the same order that doc values are ordered in. This is useful for supporting execute api in this #71374.
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 >enhancement Team:Core/Infra Meta label for core/infra team v7.13.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants