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

Adds field-caps non empty field task in the many-shards-quantitative challenge #562

Merged
merged 4 commits into from
Feb 15, 2024

Conversation

piergm
Copy link
Member

@piergm piergm commented Feb 15, 2024

Recently we added a new optional query parameter to the _field_caps API in order to
filter out fields with no values. The parameter is called include_empty_fields and
defaults to true, and if set to false it will filter out from the field_caps response all the
fields that has no value in the index. With this PR we want to benchmark this change
by adding a new task to the many-shard-quantitative challenge.

relates to: elastic/elasticsearch#103651

@piergm piergm self-assigned this Feb 15, 2024
@@ -84,5 +84,6 @@
}
},
{% include "tasks/field-caps.json" %}
Copy link
Member

Choose a reason for hiding this comment

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

You'll need a comma at the end of this line to maintain valid json

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! Fixed now :D

@gareth-ellis
Copy link
Member

One more thing, you mention its a new feature - what release will this be available in? We will need to make sure it doesnt get backported to tags for track versions that don't support it

@piergm
Copy link
Member Author

piergm commented Feb 15, 2024

@gareth-ellis it is in main now and will be available from 8.13 onwards. How do I avoid for it to be back-ported to tags before < 8.13?

@gareth-ellis
Copy link
Member

I'd misremembered, we backport changes that will work with older versions, if it won't work, then we don't need to worry. It's useful that the information is in the PR at least for future reference

Copy link
Member

@gareth-ellis gareth-ellis left a comment

Choose a reason for hiding this comment

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

LGTM

@piergm piergm merged commit fd6987e into elastic:master Feb 15, 2024
13 checks passed
inqueue pushed a commit to inqueue/rally-tracks that referenced this pull request May 14, 2024
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.

2 participants