[GitHub] [lucene-solr] pawel-bugalski-dynatrace commented on pull request #2429: LUCENE-9791 Allow calling BytesRefHash#find concurrently

2021-03-10 Thread GitBox
pawel-bugalski-dynatrace commented on pull request #2429: URL: https://github.com/apache/lucene-solr/pull/2429#issuecomment-795758595 I'm closing this pull request as it was replaced by https://github.com/apache/lucene/pull/8 that is targeting a new repository. ---

[GitHub] [lucene-solr] pawel-bugalski-dynatrace commented on pull request #2429: LUCENE-9791 Allow calling BytesRefHash#find concurrently

2021-03-04 Thread GitBox
pawel-bugalski-dynatrace commented on pull request #2429: URL: https://github.com/apache/lucene-solr/pull/2429#issuecomment-790467163 @rmuir @madrob what do you think about current state of this PR? Any more comments? What else needs to be done to merge it? ---

[GitHub] [lucene-solr] pawel-bugalski-dynatrace commented on pull request #2429: LUCENE-9791 Allow calling BytesRefHash#find concurrently

2021-02-25 Thread GitBox
pawel-bugalski-dynatrace commented on pull request #2429: URL: https://github.com/apache/lucene-solr/pull/2429#issuecomment-786053617 I've implemented some of the ideas discussed here. What do you think? This is an automated

[GitHub] [lucene-solr] pawel-bugalski-dynatrace commented on pull request #2429: LUCENE-9791 Allow calling BytesRefHash#find concurrently

2021-02-24 Thread GitBox
pawel-bugalski-dynatrace commented on pull request #2429: URL: https://github.com/apache/lucene-solr/pull/2429#issuecomment-785282649 > The problem is this change doesn't just impact find() like the description states, but also other things like add() getting called by TermsHashPerField, a