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

Fixes parent pipeline aggregations with custom metric and custom sort order #93276

Merged
merged 3 commits into from
Mar 4, 2021

Conversation

ppisljar
Copy link
Member

@ppisljar ppisljar commented Mar 2, 2021

Summary

Parent pipeline aggregations with custom metric and custom sort order on the bucket work again in visualize.

Resolves #87511

For maintainers

@ppisljar ppisljar added review Feature:Search Querying infrastructure in Kibana Feature:Aggregations Aggregation infrastructure (AggConfig, esaggs, ...) v8.0.0 Team:AppServices v7.12.0 SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week labels Mar 2, 2021
@ppisljar ppisljar requested a review from a team as a code owner March 2, 2021 17:35
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-services (Team:AppServices)

Copy link
Contributor

@streamich streamich left a comment

Choose a reason for hiding this comment

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

Code change LGTM, but there is a functional test failing.

@ppisljar
Copy link
Member Author

ppisljar commented Mar 3, 2021

@elasticmachine merge upstream

@ppisljar ppisljar added the auto-backport Deprecated - use backport:version if exact versions are needed label Mar 4, 2021
@ppisljar ppisljar merged commit a58e3bd into elastic:master Mar 4, 2021
@kibanamachine
Copy link
Contributor

💔 Backport failed

❌ 7.12: Commit could not be cherrypicked due to conflicts

To backport manually, check out the target branch and run:
node scripts/backport --pr 93276

@wylieconlon
Copy link
Contributor

@ppisljar the PR title here doesn't really fit our release note style. It doesn't tell a user what was fixed. Also, I think you need to include the 7.13.0 tag for backporting

gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 4, 2021
…into actions/terminology-api

* 'actions/terminology-api' of github.com:gmmorris/kibana:
  Make Dashboard Unsaved Changes Space Specific (elastic#92680)
  Hide Value and Funtional boost for geolocation (elastic#93683)
  [Fleet] Set all keyword and text fields for `index.query.default_field` index template setting (elastic#91791)
  can not query the world (elastic#93556)
  [Security Solutions] Sets our default date time to be "today" instead of "Last 24 hours" to enable cachability and fixes one date math bug in the URL  (elastic#93548)
  [Security Solution][Endpoint][Admin][Policy] Register as AV os restrictions tooltip note  (elastic#93306)
  fix agg config sub agg dsl generation (elastic#93276)
@ppisljar ppisljar changed the title fix agg config sub agg dsl generation Fixes parent pipeline aggregations with custom metric and custom sort order Mar 5, 2021
ppisljar added a commit to ppisljar/kibana that referenced this pull request Mar 5, 2021
# Conflicts:
#	api_docs/data.json
#	api_docs/data_search.json
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
data 813.9KB 814.0KB +45.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

ppisljar added a commit that referenced this pull request Mar 5, 2021
# Conflicts:
#	api_docs/data.json
#	api_docs/data_search.json
ppisljar added a commit to ppisljar/kibana that referenced this pull request Mar 5, 2021
# Conflicts:
#	api_docs/data.json
#	api_docs/data_search.json
ppisljar added a commit that referenced this pull request Mar 5, 2021
# Conflicts:
#	api_docs/data.json
#	api_docs/data_search.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Aggregations Aggregation infrastructure (AggConfig, esaggs, ...) Feature:Search Querying infrastructure in Kibana release_note:fix review SharedUX/fix-it-week Bugs that have been groomed and queued up for the team's next fix it week v7.12.0 v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[data.search.aggs] Sibling agg using Terms isn't able to sort the Terms agg by the metric
5 participants