honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

don't return any id when doesn't exists

This commit is contained in:
kimchy 2011-04-17 06:21:39 +03:00
parent fb35b1c993
commit efb3e97ce4
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
modules/elasticsearch/src/main/java/org/elasticsearch/action/get/TransportGetAction.java
View File

@ -248,8 +248,8 @@ public class TransportGetAction extends TransportShardSingleOperationAction<GetR
continue;
}
UidField.DocIdAndVersion docIdAndVersion = UidField.loadDocIdAndVersion(reader, uid);
// either -2 (its there, but no version associated), or an actual version
if (docIdAndVersion.docId != -1) {
// not null if it exists
if (docIdAndVersion != null) {
return docIdAndVersion;
}
}

2
modules/elasticsearch/src/main/java/org/elasticsearch/common/lucene/uid/UidField.java
View File

@ -58,7 +58,7 @@ public class UidField extends AbstractField {
try {
uid = reader.termPositions(term);
if (!uid.next()) {
return new DocIdAndVersion(Lucene.NO_DOC, -1, reader);
return null; // no doc
}
docId = uid.doc();
uid.nextPosition();

2
modules/elasticsearch/src/test/java/org/elasticsearch/common/lucene/uid/UidFieldTests.java
View File

@ -67,6 +67,6 @@ public class UidFieldTests {
writer.deleteDocuments(new Term("_uid", "1"));
reader = reader.reopen();
assertThat(UidField.loadVersion(reader, new Term("_uid", "1")), equalTo(-1l));
assertThat(UidField.loadDocIdAndVersion(reader, new Term("_uid", "1")).version, equalTo(-1l));
assertThat(UidField.loadDocIdAndVersion(reader, new Term("_uid", "1")), nullValue());
}
}