Skip to content

[Transform] Clean up internal tests#143246

Merged
prwhelan merged 4 commits intoelastic:mainfrom
prwhelan:transform-old-leak-test-thing
Mar 2, 2026
Merged

[Transform] Clean up internal tests#143246
prwhelan merged 4 commits intoelastic:mainfrom
prwhelan:transform-old-leak-test-thing

Conversation

@prwhelan
Copy link
Copy Markdown
Member

@prwhelan prwhelan commented Feb 27, 2026

Stop and delete transforms created during tests ahead of the feature
reset.

Stop and delete transforms created during tests ahead of the feature
reset.
@prwhelan prwhelan added :ml/Transform Transform Team:ML Meta label for the ML team v9.4.0 labels Feb 27, 2026
@prwhelan prwhelan closed this Feb 27, 2026
@prwhelan prwhelan reopened this Feb 27, 2026
@prwhelan prwhelan added the >test Issues or PRs that are addressing/adding tests label Feb 27, 2026
@prwhelan prwhelan changed the title transform old leak test thing [Transform] Clean up internal tests Feb 27, 2026
@prwhelan prwhelan marked this pull request as ready for review February 27, 2026 18:19
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/ml-core (Team:ML)


assertMaxPageSearchSizeInSettings(transformId, expectedMaxPageSearchSize);

stopTransform(transformId);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Generally LGTM.

Out of curiosity though - why does this test (and another one above) need both stop and delete, while all the others don't?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Some of them do not start the transform, for example testMigratedTransformIndex and testUpdateReplacesDeprecatedTransformSettings create the transform but do not start it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah I see. Thanks!

@prwhelan prwhelan enabled auto-merge (squash) March 2, 2026 16:34
@prwhelan prwhelan merged commit fe02097 into elastic:main Mar 2, 2026
32 of 35 checks passed
szybia added a commit to szybia/elasticsearch that referenced this pull request Mar 2, 2026
…locations

* upstream/main: (94 commits)
  Mute org.elasticsearch.xpack.esql.qa.mixed.EsqlClientYamlIT test {p0=esql/40_tsdb/TS Command grouping on text field} elastic#142544
  Mute org.elasticsearch.index.store.StoreDirectoryMetricsIT testDirectoryMetrics elastic#143419
  Mute org.elasticsearch.xpack.esql.qa.multi_node.GenerativeIT test elastic#143023
  TS_INFO information retrieval command (elastic#142721)
  ESQL: External source parallel execution and distribution (elastic#143349)
  Mute org.elasticsearch.index.mapper.blockloader.FlattenedFieldRootBlockLoaderTests testBlockLoaderForFieldInObject {preference=Params[syntheticSource=false, preference=DOC_VALUES]} elastic#143414
  Mute org.elasticsearch.index.mapper.blockloader.FlattenedFieldRootBlockLoaderTests testBlockLoaderForFieldInObject {preference=Params[syntheticSource=false, preference=NONE]} elastic#143413
  Mute org.elasticsearch.index.mapper.blockloader.FlattenedFieldRootBlockLoaderTests testBlockLoaderForFieldInObject {preference=Params[syntheticSource=false, preference=STORED]} elastic#143412
  Removing ingest random sampling (elastic#143289)
  Mute org.elasticsearch.xpack.esql.qa.single_node.GenerativeIT test elastic#143023
  [Transform] Clean up internal tests (elastic#143246)
  Skip time series field type merge for non-TS agg queries (elastic#143262)
  Enable zero-copy SIMD vector scoring on searchable snapshots (frozen tier) (elastic#141718)
  Mute org.elasticsearch.xpack.search.CrossClusterAsyncSearchIT testCancelViaExpirationOnRemoteResultsWithMinimizeRoundtrips elastic#143407
  Fix MemorySegmentUtilsTests (elastic#143391)
  Unmute testWorkflowsRestrictionAllowsAccess (elastic#143308)
  Cancel async query on expiry (elastic#143016)
  ESQL: Finish migrating error testing (elastic#143322)
  Reduce LuceneOperator.Status memory consumption with large QueryDSL queries (elastic#143175)
  ESQL: Generative testing with full text functions (elastic#142961)
  ...
tballison pushed a commit to tballison/elasticsearch that referenced this pull request Mar 3, 2026
Stop and delete transforms created during tests ahead of the feature
reset.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml/Transform Transform Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants