-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Open
Labels
Description
Describe the bug
This is a follow-up to Lucene change apache/lucene#12599, and OS Meta: #16934
Purpose of this issue to each of OS plugins that currently loop using RandomAccessInput.readByte, to instead use RandomAccessInput.readBytes bulk option.
For CoreSearch there was 1 instance found: #17555
Related component
Plugins
To Reproduce
- Go through plugin code
- Using IDE Find Usages for https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/store/RandomAccessInput.java readByte() method
- Check if it can be replaced with RandomAccessInput.readBytes() method.
Expected behavior
Raise PRs and call out any possible benchmark improvements.
Additional Details
Plugins
Go though all Plugins that use https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/store/RandomAccessInput.java
Screenshots
n/a
Host/Environment (please complete the following information):
- OS: n/a
- Version 3.0.0
Additional context
Add any other context about the problem here.
Reactions are currently unavailable