Skip to content

Unmute ValueSourceReaderTypeConversionTests#testLoadAll#143189

Merged
MattAlp merged 1 commit intoelastic:mainfrom
MattAlp:fix/141954-valuesource-timeout
Mar 2, 2026
Merged

Unmute ValueSourceReaderTypeConversionTests#testLoadAll#143189
MattAlp merged 1 commit intoelastic:mainfrom
MattAlp:fix/141954-valuesource-timeout

Conversation

@MattAlp
Copy link
Copy Markdown
Contributor

@MattAlp MattAlp commented Feb 26, 2026

Description

  • Remove the mute for ValueSourceReaderTypeConversionTests.testLoadAll in muted-tests.yml since myself, @ncordon, and @mouhc1ne weren't able to repro + Buildkite & history suggest it was transient / a fluke

Closes: #141954

The muted test for elastic#141954 is stable again on current main under repeated local runs, so remove the mute to restore coverage.

Made-with: Cursor
@MattAlp MattAlp added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL low-risk An open issue or test failure that is a low risk to future releases labels Feb 26, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@MattAlp MattAlp added the >test-failure Triaged test failures from CI label Feb 26, 2026
@MattAlp MattAlp requested a review from mouhc1ne February 27, 2026 13:43
@MattAlp
Copy link
Copy Markdown
Contributor Author

MattAlp commented Feb 27, 2026

I'm actually going to hold off on merging this for now, after speaking with Alex I'd like to see if we can find anything funky here by diving deeper into CI land

edit: nothing came up, time will tell if it resurfaces

@MattAlp MattAlp requested a review from alex-spies February 27, 2026 14:58
@MattAlp
Copy link
Copy Markdown
Contributor Author

MattAlp commented Feb 27, 2026

Adding @alex-spies (but please defer reviewing as much as possible, the priority is negligible)

@MattAlp MattAlp merged commit 23d4a28 into elastic:main Mar 2, 2026
35 checks passed
szybia added a commit to szybia/elasticsearch that referenced this pull request Mar 2, 2026
…cations

* upstream/main: (60 commits)
  Use batches for other bulk vector benchmarks (elastic#143167)
  Mute org.elasticsearch.xpack.esql.qa.mixed.MixedClusterEsqlSpecIT test {csv-spec:lookup-join.MvJoinKeyOnTheLookupIndexAfterStats} elastic#143388
  Mute org.elasticsearch.snapshots.ConcurrentSnapshotsIT testBackToBackQueuedDeletes elastic#143387
  [Inference API] Parse endpoint metadata from persisted endpoints (elastic#143081)
  Add cluster formation doc to DistributedArchitectureGuide (elastic#143318)
  Fix flattened root block loader null expectation (elastic#143238)
  Unmute ValueSourceReaderTypeConversionTests testLoadAll (elastic#143189)
  ESQL: Add split coalescing for many small files (elastic#143335)
  Unmute mixed-cluster spatial parse warning test (elastic#143186)
  Fix zero-size estimate in BytesRefBlock null test (elastic#143258)
  Make DataType and DataFormat top-level enums (elastic#143312)
  Add support for steps to change the target index name for later steps (elastic#142955)
  Set mayContainDuplicates flag to test deduplication (elastic#143375)
  ESQL: Fix Driver search load millis as nanos bug (elastic#143267)
  Mute org.elasticsearch.xpack.esql.qa.mixed.MixedClusterEsqlSpecIT test {csv-spec:lookup-join.LookupJoinWithMixPushableAndUnpushableFilters} elastic#143378
  ESQL: Forbid MV_EXPAND before full text functions (elastic#143249)
  ESQL: Fix unresolved name pattern (elastic#143210)
  Implement boxplot queryDSL aggregation for exponential_histograms (elastic#143026)
  Add prefetching to x64 bulk vector implementations (elastic#142387)
  Make large segment vector tests resilient to memory constraints (elastic#143366)
  ...
tballison pushed a commit to tballison/elasticsearch that referenced this pull request Mar 3, 2026
The muted test for elastic#141954 is stable again on current main under repeated local runs, so remove the mute to restore coverage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL low-risk An open issue or test failure that is a low risk to future releases Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] ValueSourceReaderTypeConversionTests testLoadAll failing

4 participants