Skip to content

WIP: Add runtime field operations to http_logs#133

Closed
cbuescher wants to merge 3 commits intoelastic:masterfrom
cbuescher:add-runtime-field
Closed

WIP: Add runtime field operations to http_logs#133
cbuescher wants to merge 3 commits intoelastic:masterfrom
cbuescher:add-runtime-field

Conversation

@cbuescher
Copy link
Copy Markdown
Member

For #elastic/elasticsearch#62621 I am doing some measurements using the new runtime fields that we would like to add to e.g. the http_logs track at some point. This is an early stage WIP mainly used for discussion so I'm opening this as a draft.

@danielmitterdorfer
Copy link
Copy Markdown
Member

@cbuescher given that Nik has added support for runtime fields in #132 I wonder whether we can close this one?

@cbuescher
Copy link
Copy Markdown
Member Author

@danielmitterdorfer thanks for the ping, I'm going to check the addition Nik made to see if this covers what I wanted to here and close if I find its covered already.

}
},
{
"name": "search-term-runtime-field",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This one is covered by in the stuff I merged a few days ago with the term operation.

},
{
"name": "runtime-field-source",
"operation-type": "search",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We do not have an operation for this one.

}
},
"aggs": {
"isGetAgg": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We don't have this one in the operations I opened either.

@DJRickyB
Copy link
Copy Markdown
Contributor

As @nik9000 has developed a pretty robust matrix of runtime fields operations in the runtime-fields challenge, I am closing this. Feel free to re-open or open an issue if there is a specific gap that should be addressed.

Thanks!
Rick B

@DJRickyB DJRickyB closed this Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants