sgup432 opened a new pull request, #16399:
URL: https://github.com/apache/lucene/pull/16399
### Description
While going through the doc values code where we check the value range for
multi valued use-case(and cardinality is not fixed), I see we end up calling 2
decode calls for each doc like below:
```
long startOffset = addresses.get(currentDoc); // decode from disk
long endOffset = addresses.get(currentDoc + 1L);
```
Seems redundant for consecutive documents. We can avoid one unnecessary
`addresses.get()` call by storing the previous offset. This is what this PR
does. Simple change.
<!--
If this is your first contribution to Lucene, please make sure you have
reviewed the contribution guide.
https://github.com/apache/lucene/blob/main/CONTRIBUTING.md
-->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
