Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check index setting for source mode in SourceOnlySnapshotRepository #116002

Merged

Conversation

kkrik-es
Copy link
Contributor

SourceOnlySnapshotRepository has checks for incomplete source that use the document mapper. This is only populated when indexing docs, so earlier checks need to check the index setting for source mode to detect if the source will (eventually) be incomplete.

This was detected as part of switching integration tests to use the index setting instead of the source mapper for configuring source mode, #115926,

@kkrik-es kkrik-es added >non-issue auto-backport Automatically create backport pull requests when merged Team:StorageEngine :StorageEngine/Mapping The storage related side of mappings v8.17.0 labels Oct 31, 2024
@kkrik-es kkrik-es self-assigned this Oct 31, 2024
@kkrik-es kkrik-es marked this pull request as ready for review October 31, 2024 12:00
@kkrik-es kkrik-es requested a review from martijnvg October 31, 2024 12:00
@kkrik-es
Copy link
Contributor Author

Test failure is unrelated, #116005

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

@kkrik-es kkrik-es merged commit 37a4ee3 into elastic:main Oct 31, 2024
16 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

The backport operation could not be completed due to the following error:

An unexpected error occurred when attempting to backport this PR.

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

kkrik-es added a commit to kkrik-es/elasticsearch that referenced this pull request Oct 31, 2024
…lastic#116002)

* Check index setting for source mode in SourceOnlySnapshotRepository

* update

* Revert "update"

This reverts commit 9bbf049.

(cherry picked from commit 37a4ee3)
@kkrik-es
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Oct 31, 2024
…116002) (#116014)

* Check index setting for source mode in SourceOnlySnapshotRepository

* update

* Revert "update"

This reverts commit 9bbf049.

(cherry picked from commit 37a4ee3)
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Nov 4, 2024
…lastic#116002)

* Check index setting for source mode in SourceOnlySnapshotRepository

* update

* Revert "update"

This reverts commit 9bbf049.
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 backport pending >non-issue :StorageEngine/Mapping The storage related side of mappings Team:StorageEngine v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants