Skip to content

Conversation

@jordan-powers
Copy link
Contributor

Follow-up to #129969 to remove the feature flag.

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@elasticsearchmachine elasticsearchmachine added the serverless-linked Added by automation, don't add manually label Aug 26, 2025
Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

One comment, LGTM otherwise.

public ShardFieldStats shardFieldStats() {
try (var searcher = acquireSearcher("shard_field_stats", Engine.SearcherScope.INTERNAL)) {
return shardFieldStats(searcher.getLeafContexts());
boolean isStateless = DiscoveryNode.isStateless(engineConfig.getIndexSettings().getNodeSettings());
Copy link
Member

Choose a reason for hiding this comment

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

Can we turn this into a field in Engine? Then there is no need to read the node settings every time this method gets invoked.

@jordan-powers jordan-powers enabled auto-merge (squash) August 27, 2025 14:47
@jordan-powers jordan-powers merged commit f846275 into elastic:main Aug 27, 2025
33 checks passed
@jordan-powers jordan-powers deleted the tracking-postings-memory-bytes-remove-ff branch September 2, 2025 18:48
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.

3 participants