uschindler commented on PR #12841: URL: https://github.com/apache/lucene/pull/12841#issuecomment-1854877951
Let's keep ByteBufferIndexInput (without s) as is. Maybe work on that later. We would need to figure out what is causing slowness here. So revert the change and copy paste code to MemorysegmentIndexInput. My question: the coded 9.9 was already released, but as there is no index format change, we can just move the code away from the codec. But as this is now part of indexinput, we must be sure that the format and API is final. I tend to think we should for now only merge this to main. Before backporting we should ensure that java 11 is not causing speed regression. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
