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] Default to mmapfs within hybridfs #9377

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport b58bec4 from #8508.

* Default to mmapfs within hybridfs

Signed-off-by: David Zane <[email protected]>

* Add index setting validation func

Signed-off-by: David Zane <[email protected]>

* Reviewer comments

Signed-off-by: David Zane <[email protected]>

* Clean up, mmap.extensions validation

Signed-off-by: David Zane <[email protected]>

* Deprecation flag, build all_ext list

Signed-off-by: David Zane <[email protected]>

* Make nioExtensions unmodifiable

Signed-off-by: David Zane <[email protected]>

---------

Signed-off-by: David Zane <[email protected]>
(cherry picked from commit b58bec4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot
Copy link
Contributor Author

Compatibility status:

Checks if related components are compatible with change 2a3e020

Incompatible components

Incompatible components: [https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/asynchronous-search.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/ml-commons.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #9377 (68aae94) into 2.x (2a3e020) will increase coverage by 0.02%.
Report is 2 commits behind head on 2.x.
The diff coverage is 86.84%.

@@             Coverage Diff              @@
##                2.x    #9377      +/-   ##
============================================
+ Coverage     70.85%   70.88%   +0.02%     
- Complexity    57540    57587      +47     
============================================
  Files          4761     4761              
  Lines        271849   271883      +34     
  Branches      40087    40093       +6     
============================================
+ Hits         192618   192723     +105     
+ Misses        62692    62672      -20     
+ Partials      16539    16488      -51     
Files Changed Coverage Δ
...pensearch/common/settings/IndexScopedSettings.java 100.00% <ø> (ø)
...rc/main/java/org/opensearch/index/IndexModule.java 82.35% <81.48%> (-0.14%) ⬇️
...org/opensearch/index/store/FsDirectoryFactory.java 79.72% <100.00%> (+2.11%) ⬆️

... and 483 files with indirect coverage changes

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

#8059

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@andrross andrross merged commit 0c49c51 into 2.x Aug 16, 2023
34 checks passed
@github-actions github-actions bot deleted the backport/backport-8508-to-2.x branch August 16, 2023 15:20
@nknize nknize mentioned this pull request Aug 28, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants