[jira] [Commented] (SOLR-8238) Make Solr respect preferredLeader at startup

2023-03-04 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/SOLR-8238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17696497#comment-17696497 ] Erick Erickson commented on SOLR-8238: -- "Me or one of my esteemed colleagues is likely to re-open

[GitHub] [solr] dsmiley commented on pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
dsmiley commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1454899402 Thanks Rob for chiming in. Made two commits. I looked at the ComplexPhraseQParser test and I don't think we should bend over backwards to support `hl.usePhraseHighlighter=false`,

[GitHub] [solr] dsmiley commented on pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
dsmiley commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1454898195 Made two commits. I looked at the ComplexPhraseQParser changes and I don't think we should bend over backwards to support hl.usePhraseHighlighter=false, which is what the failing test

[GitHub] [solr] rmuir commented on pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
rmuir commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1454886054 Creating this MultiTerms doesn't really do anything. It doesn't do any IO or anything like that. It essentially makes a list of reader.terms() for each segment. It doesn't sort this list or

[GitHub] [solr] dsmiley commented on pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
dsmiley commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1454873263 I'll be sad to see the Terms caching go in SlowCompositeReaderWrapper but maybe it has to be this way. Although I added the test of this, removed here, the underlying caching pre-dated me

[GitHub] [solr] andywebb1975 commented on pull request #1324: SOLR-16643: Add multiplying boost option to rerank query parser

2023-03-04 Thread via GitHub
andywebb1975 commented on PR #1324: URL: https://github.com/apache/solr/pull/1324#issuecomment-1454711385 I've made a new PR introducing both options: https://github.com/apache/solr/pull/1431 - could you give that a review please? (It felt cleaner to raise a new PR, though now I'm

[jira] [Updated] (SOLR-16643) Add multiply/replace options to ReRank query parser

2023-03-04 Thread Andy Webb (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andy Webb updated SOLR-16643: - Description: I noticed the ReRank query parser adds the score from the {{reRankQuery}} to the original

[jira] [Updated] (SOLR-16643) Add multiply/replace options to ReRank query parser

2023-03-04 Thread Andy Webb (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andy Webb updated SOLR-16643: - Summary: Add multiply/replace options to ReRank query parser (was: Add multiplying boost option to

[GitHub] [solr] vinayakphegde commented on a diff in pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
vinayakphegde commented on code in PR #1360: URL: https://github.com/apache/solr/pull/1360#discussion_r1125445848 ## solr/core/src/java/org/apache/solr/core/MMapDirectoryFactory.java: ## @@ -63,7 +63,7 @@ protected Directory create(String path, LockFactory lockFactory,

[GitHub] [solr] andywebb1975 opened a new pull request, #1431: SOLR-16643: Add reRankOperator=multiply/replace options to ReRank query parser

2023-03-04 Thread via GitHub
andywebb1975 opened a new pull request, #1431: URL: https://github.com/apache/solr/pull/1431 https://issues.apache.org/jira/browse/SOLR-16643 # Description Please provide a short description of the changes you're making with this pull request. # Solution

[GitHub] [solr] uschindler commented on a diff in pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
uschindler commented on code in PR #1360: URL: https://github.com/apache/solr/pull/1360#discussion_r1125444982 ## solr/core/src/java/org/apache/solr/core/MMapDirectoryFactory.java: ## @@ -63,7 +63,7 @@ protected Directory create(String path, LockFactory lockFactory, DirContext

[GitHub] [solr] mkhludnev commented on pull request #1324: SOLR-16643: Add multiplying boost option to rerank query parser

2023-03-04 Thread via GitHub
mkhludnev commented on PR #1324: URL: https://github.com/apache/solr/pull/1324#issuecomment-1454685739 Congratulations, @andywebb1975! Pushing both altogether makes more sense to me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [solr] andywebb1975 commented on pull request #1324: SOLR-16643: Add multiplying boost option to rerank query parser

2023-03-04 Thread via GitHub
andywebb1975 commented on PR #1324: URL: https://github.com/apache/solr/pull/1324#issuecomment-1454682268 I have committer rights now - a massive thank you if you were involved in arranging this! I'm wondering whether to merge this as-is, or supersede it with a version including a

[GitHub] [solr] uschindler commented on a diff in pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
uschindler commented on code in PR #1360: URL: https://github.com/apache/solr/pull/1360#discussion_r1125416874 ## solr/core/src/java/org/apache/solr/core/MMapDirectoryFactory.java: ## @@ -63,7 +63,7 @@ protected Directory create(String path, LockFactory lockFactory, DirContext

[GitHub] [solr] uschindler commented on a diff in pull request #1360: SOLR 16642 : upgrade Solr to use Lucene 9.5.0

2023-03-04 Thread via GitHub
uschindler commented on code in PR #1360: URL: https://github.com/apache/solr/pull/1360#discussion_r1125416518 ## solr/core/src/java/org/apache/solr/core/MMapDirectoryFactory.java: ## @@ -63,7 +63,7 @@ protected Directory create(String path, LockFactory lockFactory, DirContext

[jira] [Commented] (SOLR-16585) All docs query with any nonzero positive start value throws NPE with "this.docs is null"

2023-03-04 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17696417#comment-17696417 ] ASF subversion and git services commented on SOLR-16585: Commit

[GitHub] [solr] mkhludnev merged pull request #1430: SOLR-16585: examples test for match all paging (#1378)

2023-03-04 Thread via GitHub
mkhludnev merged PR #1430: URL: https://github.com/apache/solr/pull/1430 -- 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: