-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[TSDS] Reindexing seems to ignore routing_path derived from mappings #125607
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
Comments
Pinging @elastic/es-storage-engine (Team:StorageEngine) |
In step 4 does the destination data stream exist? I suspect that |
Hello @lkts At step (4), the data stream doesn't exist, but I would expect to be created automatically (I have the rights and the index template would generate a data stream). But ignoring this, you can see that at step (6) I create the destination data stream manually (7) and still the reindex fails (8). Only adding the To me, it is not normal that we have a different behavior if the |
This happens because reindexer tries to build settings before the index is created elasticsearch/modules/reindex/src/main/java/org/elasticsearch/reindex/Reindexer.java Line 262 in 9feac78
This crucially is different from Line 1047 in 9feac78
IndexSettingProvider s.
It's an issue with reindexing that is not specific to TSDS. TSDS just has a setting validation that triggers it. I am not sure if these two code paths can be unified so let's ask somebody who knows this. |
Pinging @elastic/es-data-management (Team:Data Management) |
In theory this validation can be bypassed by replacing this elasticsearch/modules/reindex/src/main/java/org/elasticsearch/reindex/Reindexer.java Line 263 in 9feac78
with
Not sure if its appropriate. |
Elasticsearch Version
8.17.4
Installed Plugins
No response
Java Version
bundled
OS Version
Not relevant
Problem Description
When attempting to reindex a TSDS data stream into another, the reindex doesn't start because of the error:
Attempting to reproduce, I noticed it triggers the error only when the
routing_path
on the destination index is not defined explicitly but deduced from the TSDS dimensions in the mappings.Steps to Reproduce
Spawn an ECH deployment 8.17.4 (but also older versions present the same issue)
GET _index_template/metrics-fleet_server.agent_status
as we want to reindex the data streamsmetrics-fleet_server.agent_status...
We create a slightly modified index template reusing the existing index template
routing_path
(GET metrics-fleet_server.agent_status-storedsource/_settings
)Try to re-execute the reindex, it fails again, same error
Update the Index template to explicitly define the
routing_path
:Logs (if relevant)
No response
The text was updated successfully, but these errors were encountered: