gbbafna commented on issue #15054: URL: https://github.com/apache/lucene/issues/15054#issuecomment-3186675138
Thanks @uschindler , @navneet1v , @onestardao @uschindler , what is the reason behind using `READONCE` ? Is there some issue with DEFAULT ? We have take heap dumps and verified that IndexInput does get closed . If there is a leak , there would be a problem even with `READONCE` as well , right ? >If you read/copy the stuff multi-threaded access is totally wrong (unless you have clones of indexinputs that do random access or similar in parallel). I'd strongly recommend to refactor your own code to use READONCE in a way like it is designed for: "like an InputStream and only from one thread". So I'd recommend to have the code possibly multi-threaded but open, read and copy and close the file in one thread @msfroh is right about it . For multipart upload, we need all threads to read the file concurrently . Can you please share the reasoning of why multi-threaded access is totally wrong ? Is there a limitation which is undocumented ? >To me this really looks like a leak in closing IndexInputs. We have take heap dumps and verified that IndexInput does get closed . -- 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
