Skip to content

[Transform] Remove tests for deleted code#145685

Merged
prwhelan merged 2 commits intoelastic:mainfrom
prwhelan:fix/141112
Apr 8, 2026
Merged

[Transform] Remove tests for deleted code#145685
prwhelan merged 2 commits intoelastic:mainfrom
prwhelan:fix/141112

Conversation

@prwhelan
Copy link
Copy Markdown
Member

@prwhelan prwhelan commented Apr 3, 2026

In #140617, we had deleted BWC code to check checkpoint info. This test pattern is no longer needed to verify that behavior.

Resolve #141112

In elastic#140617, we had deleted BWC code to check checkpoint info. This test
pattern is no longer needed to verify that behavior.

Resolve elastic#141112
@prwhelan prwhelan added >test Issues or PRs that are addressing/adding tests :ml/Transform Transform Team:ML Meta label for the ML team v9.4.0 labels Apr 3, 2026
@prwhelan prwhelan marked this pull request as ready for review April 3, 2026 17:29
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Contributor

@edsavage edsavage left a comment

Choose a reason for hiding this comment

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

LGTM

@prwhelan prwhelan enabled auto-merge (squash) April 8, 2026 13:38
@prwhelan prwhelan merged commit f9635b7 into elastic:main Apr 8, 2026
35 checks passed
szybia added a commit to szybia/elasticsearch that referenced this pull request Apr 8, 2026
* upstream/main:
  Mute org.elasticsearch.xpack.esql.expression.function.aggregate.FirstDocIdGroupingAggregatorFunctionTests testSimple elastic#145923
  Reindex relocation: store source TaskResult at destination node (elastic#145488)
  Bump versions after 9.2.8 release
  [CI] DLMFrozenTransitionServiceTests testCheckForFrozenIndicesReturnsEarlyWhenCapacityExhausted failing [elastic#145778] (elastic#145906)
  Update branches.json for 9.2.8 release
  ESQL: Clarify inheriting from Attributes (elastic#145898)
  Bump versions after 9.3.3 release
  Update branches.json for 9.3.3 release
  Prune changelogs after 8.19.14 release
  Bump versions after 8.19.14 release
  Update branches.json for 8.19.14 release
  [ML] Call old inference API (elastic#145690)
  ESQL: Unmute CsvIT sumWithOverflowRow (elastic#145893)
  Index a document when testing runtime fields shadowing dimensions & metrics (elastic#145882)
  [TEST] Fix version check in testSequenceNumbersDisabled (elastic#145879)
  [ESQL] Per-file filter pushdown awareness (elastic#145755)
  Unmute testGetReindexFollowsRelocation (elastic#145841)
  Correctly ignore system indices when validating dot-prefixed indices (elastic#128868)
  [Transform] Remove tests for deleted code (elastic#145685)
  ESQL: Add generative tests for LIMIT BY (elastic#144238)
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.

[CI] TransformCheckpointServiceNodeTests testGetCheckpointStats failing

3 participants