Skip to content

Update TransformIndexer Exception - #150083

Merged
elasticsearchmachine merged 5 commits into
elastic:mainfrom
joshua-adams-1:fix-transform-indexer
Jun 4, 2026
Merged

Update TransformIndexer Exception#150083
elasticsearchmachine merged 5 commits into
elastic:mainfrom
joshua-adams-1:fix-transform-indexer

Conversation

@joshua-adams-1

@joshua-adams-1 joshua-adams-1 commented May 28, 2026

Copy link
Copy Markdown
Contributor

There was a bug where the BulkByPaginatedSearchResponse.getDeleted() field was being passed into the exception message rather than the version conflicts

@joshua-adams-1
Update TransformIndexer Exception
c3ba16f 
There was a bug where the BulkByPaginatedSearchResponse.getDeleted()
field was being passed into the exception message rather than the
version conflicts
@joshua-adams-1 joshua-adams-1 self-assigned this May 28, 2026
@joshua-adams-1
joshua-adams-1 marked this pull request as ready for review May 28, 2026 14:22
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label May 28, 2026
@joshua-adams-1

joshua-adams-1 commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

I also don't know which team to assign this to, if you have any advice?

@DaveCTurner

DaveCTurner commented May 29, 2026

Copy link
Copy Markdown
Member

Transform is owned by the ML team. Change seems reasonable from a first glance but I don't feel I have enough context to review it myself.

@joshua-adams-1 joshua-adams-1 added the :ml Machine learning label May 29, 2026
@elasticsearchmachine elasticsearchmachine added Team:ML Meta label for the ML team and removed needs:triage Requires assignment of a team area label labels May 29, 2026
@elasticsearchmachine

elasticsearchmachine commented May 29, 2026

Copy link
Copy Markdown
Collaborator

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

@elasticsearchmachine

elasticsearchmachine commented May 29, 2026

Copy link
Copy Markdown
Collaborator

Hi @joshua-adams-1, I've created a changelog YAML for you.

@benwtrent benwtrent added :ml/Transform Transform and removed :ml Machine learning labels May 29, 2026
prwhelan
prwhelan approved these changes Jun 3, 2026
View reviewed changes

@prwhelan prwhelan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

if you feel it is a bug, please backport it to 8.19 and all active 9.x branches

@joshua-adams-1 joshua-adams-1 added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged labels Jun 4, 2026
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

⏳ Building and deploying preview... View progress

This comment will be updated with preview links when the build is complete.

@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

ℹ️ Important: Docs version tagging

👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version.

We use applies_to tags to mark version-specific features and changes.

Expand for a quick overview

When to use applies_to tags:

✅ At the page level to indicate which products/deployments the content applies to (mandatory)
✅ When features change state (e.g. preview, ga) in a specific version
✅ When availability differs across deployments and environments

What NOT to do:

❌ Don't remove or replace information that applies to an older version
❌ Don't add new information that applies to a specific version without an applies_to tag
❌ Don't forget that applies_to tags can be used at the page, section, and inline level

🤔 Need help?

@elasticsearchmachine
elasticsearchmachine merged commit ca49099 into elastic:main Jun 4, 2026
36 checks passed
@joshua-adams-1
joshua-adams-1 deleted the fix-transform-indexer branch June 4, 2026 11:18
@elasticsearchmachine

elasticsearchmachine commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
❌ 8.19 Commit could not be cherrypicked due to conflicts
❌ 9.4 Commit could not be cherrypicked due to conflicts
❌ 9.3 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 150083

@joshua-adams-1

joshua-adams-1 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

Backported to:

valeriy42 pushed a commit to valeriy42/elasticsearch that referenced this pull request Jun 18, 2026
@joshua-adams-1 @valeriy42
Update TransformIndexer Exception (elastic#150083)
012c3b9 
There was a bug where the `BulkByPaginatedSearchResponse.getDeleted()`
field was being passed into the exception message rather than the
version conflicts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@prwhelan prwhelan prwhelan approved these changes
@DaveCTurner DaveCTurner Awaiting requested review from DaveCTurner

Assignees

@joshua-adams-1 joshua-adams-1

Labels

auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport pending >bug :ml/Transform Transform Team:ML Meta label for the ML team v8.19.17 v9.4.3 v9.5.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@joshua-adams-1 @DaveCTurner @elasticsearchmachine @prwhelan @benwtrent