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

Comments

Projects
None yet
1 participant
@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 referenced this issue Oct 13, 2013

Closed

Fix payloads tv #3893

brwe added a commit to brwe/elasticsearch that referenced this issue Oct 14, 2013

fix bug in term vector api, payloads were not handles correctly when …
…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 in 719d1e0 Oct 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.