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

[Backport 2.x] Refactor async blob read to avoid blocking calls, support non multipa… #10278

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 61495bf from #10192.

…rt calls (#10192)

Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
(cherry picked from commit 61495bf)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Compatibility status:

Checks if related components are compatible with change e8d107d

Incompatible components

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

Gradle Check (Jenkins) Run Completed with:

#10006

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.remotestore.multipart.RemoteStoreMultipartIT.testRestoreSnapshotToIndexWithSameNameDifferentUUID

@codecov
Copy link

codecov bot commented Oct 3, 2023

Codecov Report

Merging #10278 (e8d107d) into 2.x (eaccdf5) will decrease coverage by 0.01%.
Report is 13 commits behind head on 2.x.
The diff coverage is 73.45%.

@@             Coverage Diff              @@
##                2.x   #10278      +/-   ##
============================================
- Coverage     70.87%   70.87%   -0.01%     
- Complexity    58344    58345       +1     
============================================
  Files          4814     4814              
  Lines        275527   275603      +76     
  Branches      40503    40519      +16     
============================================
+ Hits         195280   195321      +41     
+ Misses        63613    63579      -34     
- Partials      16634    16703      +69     
Files Coverage Δ
...main/java/org/opensearch/core/rest/RestStatus.java 97.33% <100.00%> (+8.14%) ⬆️
...arch/analysis/common/MappingCharFilterFactory.java 50.00% <100.00%> (ø)
...n/java/org/opensearch/index/analysis/Analysis.java 84.21% <100.00%> (-0.12%) ⬇️
.../opensearch/index/shard/InternalIndexingStats.java 87.50% <100.00%> (ø)
...in/java/org/opensearch/indices/IndicesService.java 70.07% <100.00%> (-0.64%) ⬇️
...rg/opensearch/action/bulk/TransportBulkAction.java 76.44% <94.44%> (+0.37%) ⬆️
...rg/opensearch/repositories/s3/S3BlobContainer.java 77.18% <76.66%> (-1.02%) ⬇️
...java/org/opensearch/index/shard/IndexingStats.java 86.61% <81.81%> (+2.30%) ⬆️
...pensearch/action/update/TransportUpdateAction.java 4.24% <0.00%> (-0.34%) ⬇️

... and 461 files with indirect coverage changes

@kotwanikunal kotwanikunal merged commit a0f9736 into 2.x Oct 3, 2023
39 of 64 checks passed
@github-actions github-actions bot deleted the backport/backport-10192-to-2.x branch October 3, 2023 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants