honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-08 22:14:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add new extensions for Lucene86 points codec to FsDirectoryFactory (#58226) (#58233)

Browse Source
This commit is contained in:
Ignacio Vera 2020-06-17 12:55:33 +02:00 committed by GitHub
parent 36dbf08d47
commit b6585f2b51
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
server/src/main/java/org/elasticsearch/index/store/FsDirectoryFactory.java
View File

@ -160,7 +160,10 @@ public class FsDirectoryFactory implements IndexStorePlugin.DirectoryFactory {
// We want to open the terms index and KD-tree index off-heap to save memory, but this only performs // We want to open the terms index and KD-tree index off-heap to save memory, but this only performs
// well if using mmap. // well if using mmap.
case "tip": case "tip":
// dim files only apply up to lucene 8.x indices. It can be removed once we are in lucene 10
case "dim": case "dim":
case "kdd":
case "kdi":
// Compound files are tricky because they store all the information for the segment. Benchmarks // Compound files are tricky because they store all the information for the segment. Benchmarks
// suggested that not mapping them hurts performance. // suggested that not mapping them hurts performance.
case "cfs": case "cfs":

2
server/src/test/java/org/elasticsearch/index/store/FsDirectoryFactoryTests.java
View File

@ -64,6 +64,8 @@ public class FsDirectoryFactoryTests extends ESTestCase {
assertTrue(hybridDirectory.useDelegate("foo.tip")); assertTrue(hybridDirectory.useDelegate("foo.tip"));
assertTrue(hybridDirectory.useDelegate("foo.cfs")); assertTrue(hybridDirectory.useDelegate("foo.cfs"));
assertTrue(hybridDirectory.useDelegate("foo.dim")); assertTrue(hybridDirectory.useDelegate("foo.dim"));
assertTrue(hybridDirectory.useDelegate("foo.kdd"));
assertTrue(hybridDirectory.useDelegate("foo.kdi"));
assertFalse(hybridDirectory.useDelegate("foo.bar")); assertFalse(hybridDirectory.useDelegate("foo.bar"));
MMapDirectory delegate = hybridDirectory.getDelegate(); MMapDirectory delegate = hybridDirectory.getDelegate();
assertThat(delegate, Matchers.instanceOf(FsDirectoryFactory.PreLoadMMapDirectory.class)); assertThat(delegate, Matchers.instanceOf(FsDirectoryFactory.PreLoadMMapDirectory.class));