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

DistributorDirectory shouldn't search for directory when reading existing file #7323

Closed
wants to merge 1 commit into from

Conversation

mikemccand
Copy link
Contributor

I just changed the logic in the private getDirectory() to only call distributor.any() if there wasn't already a binding for the requested file name.

Closes #7306

@bleskes
Copy link
Contributor

bleskes commented Aug 19, 2014

LGTM. Just needs some labels..

@mikemccand
Copy link
Contributor Author

Thanks @bleskes I'll commit ... on the labels, I think we are only supposed to label either the issue or the PR but not both, else it causes confusion when generating release notes? I labeled the issue (#7306) in this case ... though I should have put the review label on this one.

mikemccand added a commit that referenced this pull request Aug 19, 2014
…g existing file

This was causing too much work e.g. when pulling node stats or when
opening a new reader, because the least_used distributor would
unnecessarily check free disk space on all path.data entires every
time we try to open a file for reading or check its length.

Closes #7306

Closes #7323
mikemccand added a commit that referenced this pull request Aug 19, 2014
…g existing file

This was causing too much work e.g. when pulling node stats or when
opening a new reader, because the least_used distributor would
unnecessarily check free disk space on all path.data entires every
time we try to open a file for reading or check its length.

Closes #7306

Closes #7323
mikemccand added a commit that referenced this pull request Sep 8, 2014
…g existing file

This was causing too much work e.g. when pulling node stats or when
opening a new reader, because the least_used distributor would
unnecessarily check free disk space on all path.data entires every
time we try to open a file for reading or check its length.

Closes #7306

Closes #7323
@clintongormley clintongormley changed the title Core: DistributorDirectory shouldn't search for directory when reading existing file Internal: DistributorDirectory shouldn't search for directory when reading existing file Sep 8, 2014
@clintongormley clintongormley changed the title Internal: DistributorDirectory shouldn't search for directory when reading existing file DistributorDirectory shouldn't search for directory when reading existing file Jun 7, 2015
@clintongormley clintongormley added :Distributed/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. >bug and removed >enhancement labels Jun 7, 2015
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
…g existing file

This was causing too much work e.g. when pulling node stats or when
opening a new reader, because the least_used distributor would
unnecessarily check free disk space on all path.data entires every
time we try to open a file for reading or check its length.

Closes elastic#7306

Closes elastic#7323
@clintongormley clintongormley added :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. and removed :Distributed/Store Issues around managing unopened Lucene indices. If it touches Store.java, this is a likely label. labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v1.3.3 v1.4.0.Beta1 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal: DistributorDirectory should not invoke distributor when reading an existing file
3 participants