-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Backport 2.x] [Searchable Snapshot] Fix bug of Searchable Snapshot Dependency on repository chunk_size #12486
Conversation
…pository chunk_size (#12277) * implement logic of fetching blocks from multiple chunks of snapshot file. Signed-off-by: Rishikesh1159 <[email protected]> * Refactor and address comments. Signed-off-by: Rishikesh1159 <[email protected]> * apply spotless check Signed-off-by: Rishikesh1159 <[email protected]> * Address comments of using a different data structure to fetch blob parts. Signed-off-by: Rishikesh1159 <[email protected]> * remove unnecessary code. Signed-off-by: Rishikesh1159 <[email protected]> * Refactor outputstream usage. Signed-off-by: Rishikesh1159 <[email protected]> * refactor blobpart logic into a separate method and add unit tests. Signed-off-by: Rishikesh1159 <[email protected]> * Add new unit tests. Signed-off-by: Rishikesh1159 <[email protected]> --------- Signed-off-by: Rishikesh1159 <[email protected]> (cherry picked from commit 3125b94) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Compatibility status:Checks if related components are compatible with change f8368b0 Incompatible componentsSkipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/performance-analyzer.git] |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #12486 +/- ##
============================================
+ Coverage 71.14% 71.24% +0.10%
- Complexity 59957 60012 +55
============================================
Files 4945 4945
Lines 282335 282349 +14
Branches 41366 41369 +3
============================================
+ Hits 200870 201166 +296
+ Misses 64447 64096 -351
- Partials 17018 17087 +69 ☔ View full report in Codecov by Sentry. |
❌ Gradle check result for f8368b0: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
❕ Gradle check result for f8368b0: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Backport 3125b94 from #12277.