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

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

[GitHub] [solr] stillalex commented on pull request #1357: SOLR-16540 Fix errors importing main and branch_9x into eclipse

2023-03-03 Thread via GitHub
stillalex commented on PR #1357: URL: https://github.com/apache/solr/pull/1357#issuecomment-1454324266 @gerlowskija do you want to mark SOLR-16540 as resolved too? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [solr] stillalex commented on a diff in pull request #1410: SOLR-16676 Http2SolrClient loss of MDC context

2023-03-03 Thread via GitHub
stillalex commented on code in PR #1410: URL: https://github.com/apache/solr/pull/1410#discussion_r1125194334 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java: ## @@ -1273,4 +1287,33 @@ static SslContextFactory.Client getDefaultSslContextFactory() {

[GitHub] [solr] stillalex commented on a diff in pull request #1410: SOLR-16676 Http2SolrClient loss of MDC context

2023-03-03 Thread via GitHub
stillalex commented on code in PR #1410: URL: https://github.com/apache/solr/pull/1410#discussion_r1125193069 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/Http2SolrClientTest.java: ## @@ -469,6 +474,43 @@ public void testGetById() throws Exception { } } +

[GitHub] [solr] stillalex commented on a diff in pull request #1410: SOLR-16676 Http2SolrClient loss of MDC context

2023-03-03 Thread via GitHub
stillalex commented on code in PR #1410: URL: https://github.com/apache/solr/pull/1410#discussion_r1125191417 ## solr/core/src/test/org/apache/solr/handler/TestHttpRequestId.java: ## @@ -0,0 +1,143 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

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

2023-03-03 Thread via GitHub
dsmiley commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1454217841 TestComplexPhraseQParserPlugin: I anticipated this failure. https://github.com/apache/lucene/pull/11807 This weekend I'll see if I could fix it. I'm going to look into overriding getHi

[GitHub] [solr] dsmiley commented on a diff in pull request #1410: SOLR-16676 Http2SolrClient loss of MDC context

2023-03-03 Thread via GitHub
dsmiley commented on code in PR #1410: URL: https://github.com/apache/solr/pull/1410#discussion_r1124950966 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java: ## @@ -1273,4 +1287,33 @@ static SslContextFactory.Client getDefaultSslContextFactory() {

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

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

[GitHub] [solr] dsmiley commented on a diff in pull request #1182: SOLR-16504 Convert CLI tools to Jetty HTTP 2 client.

2023-03-03 Thread via GitHub
dsmiley commented on code in PR #1182: URL: https://github.com/apache/solr/pull/1182#discussion_r1124880964 ## solr/core/src/java/org/apache/solr/packagemanager/DefaultPackageRepository.java: ## @@ -101,16 +106,21 @@ public Path download(String artifactName) throws SolrExceptio

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

2023-03-03 Thread via GitHub
mkhludnev opened a new pull request, #1430: URL: https://github.com/apache/solr/pull/1430 https://issues.apache.org/jira/browse/SOLR-16585 -- 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

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

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

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

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

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

2023-03-03 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-8238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696260#comment-17696260 ] David Smiley commented on SOLR-8238: bq. What if the node is out of commission permane

[GitHub] [solr] epugh commented on pull request #1182: SOLR-16504 Convert CLI tools to Jetty HTTP 2 client.

2023-03-03 Thread via GitHub
epugh commented on PR #1182: URL: https://github.com/apache/solr/pull/1182#issuecomment-1453825905 This is really exciting work! I did a bit of poking, and we have these integration tests in `./solr/packaging/test/` that use BATS for testing the SolrCLI. I wonder if this is the right pl

[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #1182: SOLR-16504 Convert CLI tools to Jetty HTTP 2 client.

2023-03-03 Thread via GitHub
sonatype-lift[bot] commented on code in PR #1182: URL: https://github.com/apache/solr/pull/1182#discussion_r1124750911 ## solr/core/src/java/org/apache/solr/util/SolrCLI.java: ## @@ -648,164 +619,30 @@ public static NamedList postJsonToSolr( return solrClient.request(req);

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

2023-03-03 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/SOLR-8238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696219#comment-17696219 ] Erick Erickson commented on SOLR-8238: -- Yes, I was describing the background for pref

[GitHub] [solr] bszabo97 commented on pull request #1182: SOLR-16504 Convert CLI tools to Jetty HTTP 2 client.

2023-03-03 Thread via GitHub
bszabo97 commented on PR #1182: URL: https://github.com/apache/solr/pull/1182#issuecomment-1453772240 Sorry that an update from me took so long, I was pretty stuck on a problem regarding the package manager. Now I have uploaded a new commit which resolved that problem and also remove

[GitHub] [solr] stillalex commented on pull request #1410: SOLR-16676 Http2SolrClient loss of MDC context

2023-03-03 Thread via GitHub
stillalex commented on PR #1410: URL: https://github.com/apache/solr/pull/1410#issuecomment-1453570691 deadlock avoided by setting corePoolSzize to 3. thanks @risdenk! waiting for test results but PR should be open for review again. -- This is an automated message from the Apache Git Serv

[jira] [Commented] (SOLR-16289) [interleaving] transformer does not work in SolrCloud

2023-03-03 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696125#comment-17696125 ] Christine Poerschke commented on SOLR-16289: Picking up here from the [https

[jira] [Resolved] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

2023-03-03 Thread Bruno Roustant (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16438?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruno Roustant resolved SOLR-16438. --- Resolution: Abandoned > Shard split should be able to set preferred leaders on other replicas

[jira] [Comment Edited] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

2023-03-03 Thread Bruno Roustant (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696108#comment-17696108 ] Bruno Roustant edited comment on SOLR-16438 at 3/3/23 10:07 AM: ---

[jira] [Reopened] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

2023-03-03 Thread Bruno Roustant (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16438?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bruno Roustant reopened SOLR-16438: --- > Shard split should be able to set preferred leaders on other replicas > ---

[jira] [Commented] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

2023-03-03 Thread Bruno Roustant (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16438?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17696108#comment-17696108 ] Bruno Roustant commented on SOLR-16438: --- Fully reverted these PRs (the initial and

[jira] [Commented] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

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

[jira] [Commented] (SOLR-16438) Shard split should be able to set preferred leaders on other replicas

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

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

2023-03-03 Thread via GitHub
vinayakphegde commented on PR #1360: URL: https://github.com/apache/solr/pull/1360#issuecomment-1453164773 tried to address all the issues/comments except "performance degradation for any query that isn't using timeouts" mentioned by @dsmiley now, only `org.apache.solr.search.TestComplex