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

Get term vector api broken for missing payloads #3873

Closed
brwe opened this issue Oct 10, 2013 · 0 comments
Closed

Get term vector api broken for missing payloads #3873

brwe opened this issue Oct 10, 2013 · 0 comments
Assignees

Comments

@brwe
Copy link
Contributor

brwe commented Oct 10, 2013

If not all tokens in a field have a payload, the java api for term vectors will return the payload of the previous token at that position if there was one.

For example, suppose a field only contains two tokens each occurring once, the first having a payload and the second not, then for the second token, the payload of the first would be returned.

@ghost ghost assigned brwe Oct 10, 2013
brwe added a commit to brwe/elasticsearch that referenced this issue Oct 10, 2013
@brwe brwe mentioned this issue Oct 13, 2013
brwe added a commit to brwe/elasticsearch that referenced this issue Oct 14, 2013
…some where missing

The array holding the payloads (TermVectorFields.payloads) is reused for each token. If the
previous token had payloads but the current token had not, then the payloads of the previous
token were returned, because the payloads of the previous token were never invalidated.
For example, for a field only contained two tokens each occurring once, the first having a
payload and the second not, then for the second token, the payload of the first was returned.

closes elastic#3873
@brwe brwe closed this as completed in 719d1e0 Oct 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant