Skip to content

Conversation

gmarouli
Copy link
Contributor

@gmarouli gmarouli commented Aug 1, 2025

Both test failed because the test was indexing very large documents and triggered the indexing pressure monitor. We considered adding throttling but we did not succeed so we reduced max the number of fields a document can have. This fixed the issue.

Fixes #132225
Fixes #132226

@gmarouli gmarouli added >test Issues or PRs that are addressing/adding tests :StorageEngine/Logs You know, for Logs labels Aug 1, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@gmarouli gmarouli changed the title Reduce the number of fields per document Test fix: reduce the number of fields per document in StandardVersusLogsIndexModeChallengeRestIT Aug 1, 2025
@gmarouli gmarouli added auto-backport Automatically create backport pull requests when merged v8.19.1 v9.1.1 v9.0.5 labels Aug 1, 2025
Copy link
Contributor

@parkertimmins parkertimmins left a comment

Choose a reason for hiding this comment

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

LGTM

@gmarouli gmarouli merged commit fcf0408 into elastic:main Aug 4, 2025
33 checks passed
@gmarouli gmarouli deleted the fix-indexing-pressure-in-logsdb-reindex-test branch August 4, 2025 09:06
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Aug 4, 2025

💚 Backports

Status Branch Result
9.1
8.19
9.0
gmarouli added a commit to gmarouli/elasticsearch that referenced this pull request Aug 4, 2025
gmarouli added a commit to gmarouli/elasticsearch that referenced this pull request Aug 4, 2025
gmarouli added a commit to gmarouli/elasticsearch that referenced this pull request Aug 4, 2025
szybia added a commit to szybia/elasticsearch that referenced this pull request Aug 5, 2025
…cking * upstream/main: (26 commits) [Fleet] add privileges to `kibana_system` to read integrations data (elastic#132400) Add `TestEntitlementsRule` with support for dynamic entitled node paths for testing (elastic#132077) Reduce logging frequency for GCS per project clients (elastic#132429) Skip update/100_synthetic_source tests in yamlRestCompatTests (elastic#132296) Correct exception for missing nested path (elastic#132408) Fixing esql release tests elastic#132369 (elastic#132406) Adjust date docvalue formatting to return 4xx instead of 5xx (elastic#132414) Handle nested fields with the termvectors REST API in artificial docs (elastic#92568) Only collect bulk scored vectors when exceeding min competitive (elastic#132293) Fix release tests diskbbq update (elastic#132405) ESQL: Fix skipping of generative tests (elastic#132390) Short circuit failure handling in OIDC flow (elastic#130618) Small optimization in OptimizedScalarQuantizer by using mul instead of div (elastic#132397) Aggs: Add validation to Bucket script pipeline agg (elastic#132320) ESQL: Multiple parameters in ungrouped aggs (elastic#132375) ESQL: Explain test operators (elastic#132374) EQL: Deal with internally created IN in a different way for EQL (elastic#132167) Speed up hierarchical k-means by computing distances in bulk (elastic#132384) Reduce the number of fields per document (elastic#132322) Assert current thread in ESQL (elastic#132324) ...
gmarouli added a commit to gmarouli/elasticsearch that referenced this pull request Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :StorageEngine/Logs You know, for Logs Team:StorageEngine >test Issues or PRs that are addressing/adding tests v8.19.1 v9.0.5 v9.1.1 v9.2.0

3 participants