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

mmap dim files in HybridDirectory #49272

Merged
merged 1 commit into from
Nov 19, 2019
Merged

mmap dim files in HybridDirectory #49272

merged 1 commit into from
Nov 19, 2019

Conversation

jdconrad
Copy link
Contributor

This change mmaps dim files in HybridDirectory to take advantage of off-heap BKD trees. This is based off of (#48509) via (https://issues.apache.org/jira/browse/LUCENE-8932).

Should this be backported prior to 8.0?

@jdconrad jdconrad added >enhancement :Distributed/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. v8.0.0 labels Nov 18, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Store)

Copy link
Contributor

@jpountz jpountz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds good to me and I'd backport to 7.x, but I'm curious what @danielmitterdorfer thinks.

Copy link
Member

@danielmitterdorfer danielmitterdorfer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I agree with @jpountz to backport to 7.x as well.

@jdconrad
Copy link
Contributor Author

@jpountz @danielmitterdorfer Thanks for the reviews. Will add to 7.x as well.

@jdconrad jdconrad merged commit 8e2a23a into elastic:master Nov 19, 2019
jdconrad added a commit that referenced this pull request Nov 19, 2019
This change mmaps dim files in HybridDirectory to take advantage of off-
heap BKD trees. This is based off of (#48509) via 
(https://issues.apache.org/jira/browse/LUCENE-8932).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. >enhancement v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants