zacharymorn commented on pull request #2052: URL: https://github.com/apache/lucene-solr/pull/2052#issuecomment-723400516
> Thank you @zacharymorn! It's so nice we can move this to pure java now -- this makes it much easier and lower risk for Lucene users to test. > > I suspect for Lucene applications that do both indexing and searching on a single box (e.g. Elasticsearch), this might be a big improvement in preventing large merges from hurting concurrent queries. No problem @mikemccand ! I'm glad to be able to contribute as well. One thing I notice while working on this, and also related to your comment is, it seems Lucene core doesn't seems to use this facility via any configuration at all (no reference to `NativeUnixDirectory`). So I'm wondering if it makes sense to have a follow up task to add some kind of configuration (through Merge policy?) to use it as well once the changes stabilize ? ---------------------------------------------------------------- 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. 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
