Do not attempt marking store as corrupted if the check is rejected due to shutdown#145209
Conversation
pxsalehi
added
>bug
:Distributed/Distributed
|
Pinging @elastic/es-distributed (Team:Distributed) |
|
Hi @pxsalehi, I've created a changelog YAML for you. |
🔍 Preview links for changed docs⏳ Building and deploying preview... View progress This comment will be updated with preview links when the build is complete. |
ℹ️ 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 overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
|
Serverless failure was ES|QL generative fork IT! |
…rics * upstream/main: (428 commits) ESQL: DS: Add inference/RERANK tests (elastic#145229) Unmute MMR logical plan test (elastic#145311) Do not attempt marking store as corrupted if the check is rejected due to shutdown (elastic#145209) feat(tsdb): add pipeline runtime and rename stage interfaces (elastic#145175) Fix UnresolvedException on PromQL by(step) grouping (elastic#145307) ES|QL: Optimize MMR by reducing cache size and lookup (elastic#145014) Prometheus labels/series APIs: support multiple match[] selectors (elastic#145298) Move ClientScrollablePaginatedHitSource into Reindex Module (elastic#144100) mute test class for elastic#145277 CPS mode for ViewResolver (elastic#145219) [ESQL] Disables GroupedTopNBenchmark temporarily (elastic#145124) Make exponential_histogram the default histogram type for HTTP OTLP endpoint (elastic#145065) More tests requiring an explicit confidence interval (elastic#145232) ES|QL: Adding `USER_AGENT` command (elastic#144384) ESQL: enable Generative IT after more fixes (elastic#145112) Rework FieldMapper parameter tests to not use merge builders (elastic#145213) [ESQL] Fix ORC type support gaps (elastic#145074) [Test] Unmute FollowingEngineTests.testProcessOnceOnPrimary (elastic#145192) Add PrometheusSeriesRestAction for /_prometheus/api/v1/series endpoint (elastic#144494) Prometheus labels API: add rest action (elastic#144952) ...
…e to shutdown (elastic#145209) This is kind of similar to previous fixes related to this test (e.g. elastic#122006) where restarting nodes cause exceptions that leads to the upper layers during recovery thinking the data is corrupted as we are not able to read from/via the cache.
4 participants
This is kind of similar to previous fixes related to this test (e.g. #122006) where restarting nodes cause exceptions that leads to the upper layers during recovery thinking the data is corrupted as we are not able to read from/via the cache.
I've extended the existing check in
checkIndex. Although I can imagine this can be done also in lower levels by catching the exception and wrapping it in AlreadyCloseException. Although in that case this might trickle down to more places, while here it seems rather straight-forward.Closes #123773