[jira] [Created] (SOLR-17233) The parameters of q.op are recommended to be trim()

2024-04-15 Thread ichar (Jira)
ichar created SOLR-17233: Summary: The parameters of q.op are recommended to be trim() Key: SOLR-17233 URL: https://issues.apache.org/jira/browse/SOLR-17233 Project: Solr Issue Type: Improvement

Re: [PR] SOLR-16866: Deleting a non-existent directory should not fail [solr]

2024-04-15 Thread via GitHub
HoustonPutman commented on PR #2336: URL: https://github.com/apache/solr/pull/2336#issuecomment-2057545040 Closing in favor of https://github.com/apache/solr/pull/2349 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] SOLR-16866: Deleting a non-existent directory should not fail [solr]

2024-04-15 Thread via GitHub
HoustonPutman closed pull request #2336: SOLR-16866: Deleting a non-existent directory should not fail URL: https://github.com/apache/solr/pull/2336 -- 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 t

Re: [PR] SOLR-16866: Reorder nested directory deletions to avoid NoSuchFileException [solr]

2024-04-15 Thread via GitHub
magibney commented on code in PR #2349: URL: https://github.com/apache/solr/pull/2349#discussion_r1566213067 ## solr/core/src/test/org/apache/solr/core/CachingDirectoryFactoryTest.java: ## @@ -46,6 +57,90 @@ private static class Tracker { Directory dir; } + @Test + p

Re: [PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
gus-asf commented on code in PR #2403: URL: https://github.com/apache/solr/pull/2403#discussion_r1566112815 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -621,46 +684,72 @@ public void handleRequestBody(SolrQueryRequest req, SolrQueryResponse

Re: [PR] SOLR-14763 SolrJ HTTP/2 Async API using CompletableFuture (update for 2024) [solr]

2024-04-15 Thread via GitHub
jdyer1 commented on PR #2402: URL: https://github.com/apache/solr/pull/2402#issuecomment-2057301844 I would like to merge this tomorrow, and also include with 9.6.0, unless someone says it is too late. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
gus-asf commented on code in PR #2403: URL: https://github.com/apache/solr/pull/2403#discussion_r1566094054 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -621,46 +684,72 @@ public void handleRequestBody(SolrQueryRequest req, SolrQueryResponse

Re: [PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
gus-asf commented on code in PR #2403: URL: https://github.com/apache/solr/pull/2403#discussion_r1566090370 ## solr/core/src/java/org/apache/solr/search/QueryLimits.java: ## @@ -108,12 +110,21 @@ public String formatExceptionMessage(String label) { * @throws QueryLimitsExcee

[jira] [Commented] (SOLR-14763) SolrJ Client Async HTTP/2 Requests

2024-04-15 Thread James Dyer (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17837343#comment-17837343 ] James Dyer commented on SOLR-14763: --- [PR 2402|https://github.com/apache/solr/pull/2402]

[jira] [Updated] (SOLR-14763) SolrJ Client Async HTTP/2 Requests

2024-04-15 Thread James Dyer (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14763?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James Dyer updated SOLR-14763: -- Fix Version/s: main (10.0) 9.6 > SolrJ Client Async HTTP/2 Requests > --

[jira] [Assigned] (SOLR-14763) SolrJ Client Async HTTP/2 Requests

2024-04-15 Thread James Dyer (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14763?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James Dyer reassigned SOLR-14763: - Assignee: James Dyer > SolrJ Client Async HTTP/2 Requests > -- >

Re: [PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
sigram commented on code in PR #2403: URL: https://github.com/apache/solr/pull/2403#discussion_r1566013026 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -621,46 +684,72 @@ public void handleRequestBody(SolrQueryRequest req, SolrQueryResponse

[PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
gus-asf opened a new pull request, #2403: URL: https://github.com/apache/solr/pull/2403 https://issues.apache.org/jira/browse/SOLR-17151 -- 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 sp

Re: [PR] SOLR-17151 - stop processing components once we have exceeded a query limit [solr]

2024-04-15 Thread via GitHub
gus-asf commented on PR #2403: URL: https://github.com/apache/solr/pull/2403#issuecomment-2057147058 Still needs unit test... -- 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.

Re: [PR] SOLR-17158 Terminate distributed processing quickly when query limit is reached - Initial impl [solr]

2024-04-15 Thread via GitHub
sigram commented on code in PR #2379: URL: https://github.com/apache/solr/pull/2379#discussion_r1565966780 ## solr/core/src/java/org/apache/solr/core/SolrXmlConfig.java: ## @@ -384,6 +384,9 @@ private static NodeConfig fillSolrSection(NodeConfig.NodeConfigBuilder builder,

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-04-15 Thread via GitHub
kotman12 commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1565959687 ## solr/modules/monitor/src/test-files/solr/collection1/solrconfig-no-cache.xml: ## @@ -0,0 +1,185 @@ + + + + + 9.4 Review Comment: thanks for catching this! .. I ha

Re: [PR] SOLR-16866: Reorder nested directory deletions to avoid NoSuchFileException [solr]

2024-04-15 Thread via GitHub
HoustonPutman commented on code in PR #2349: URL: https://github.com/apache/solr/pull/2349#discussion_r1565912035 ## solr/core/src/test/org/apache/solr/core/CachingDirectoryFactoryTest.java: ## @@ -46,6 +57,90 @@ private static class Tracker { Directory dir; } + @Test

Re: [PR] SolrJ HTTP/2 Async API using CompletableFuture (update for 2024) [solr]

2024-04-15 Thread via GitHub
jdyer1 commented on code in PR #2402: URL: https://github.com/apache/solr/pull/2402#discussion_r1565857878 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/Http2SolrClientTest.java: ## @@ -299,7 +299,9 @@ public void testAsyncException() throws Exception { public void

Re: [PR] SolrJ HTTP/2 Async API using CompletableFuture (update for 2024) [solr]

2024-04-15 Thread via GitHub
jdyer1 commented on code in PR #2402: URL: https://github.com/apache/solr/pull/2402#discussion_r1565855347 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/LBHttp2SolrClientTest.java: ## @@ -51,4 +67,249 @@ public void testLBHttp2SolrClientWithTheseParamNamesInTheUrl()

[jira] [Commented] (SOLR-17018) LTR queries producing out of memory issues

2024-04-15 Thread Alessandro Benedetti (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17837246#comment-17837246 ] Alessandro Benedetti commented on SOLR-17018: - The bugfix has been also backp

[jira] [Updated] (SOLR-17018) LTR queries producing out of memory issues

2024-04-15 Thread Alessandro Benedetti (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17018?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alessandro Benedetti updated SOLR-17018: Fix Version/s: 9.1.2 > LTR queries producing out of memory issues > ---

[jira] [Commented] (SOLR-17018) LTR queries producing out of memory issues

2024-04-15 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17018?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17837242#comment-17837242 ] ASF subversion and git services commented on SOLR-17018: Commit 1

Re: [PR] Fix/solr 17018 branch 9 1 [solr]

2024-04-15 Thread via GitHub
alessandrobenedetti merged PR #2358: URL: https://github.com/apache/solr/pull/2358 -- 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...@so

Re: [PR] SOLR-13350: Multithreaded search [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on code in PR #2248: URL: https://github.com/apache/solr/pull/2248#discussion_r1565699615 ## solr/core/src/java/org/apache/solr/search/MultiThreadedSearcher.java: ## @@ -0,0 +1,415 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more

Re: [PR] SOLR-13350: Multithreaded search [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on code in PR #2248: URL: https://github.com/apache/solr/pull/2248#discussion_r1565673302 ## solr/core/src/java/org/apache/solr/search/MultiThreadedSearcher.java: ## @@ -0,0 +1,415 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more

Re: [PR] SOLR-13350: Multithreaded search [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on PR #2248: URL: https://github.com/apache/solr/pull/2248#issuecomment-2056670213 > Definitely the segment bit sets need to be sized to the segment, plus no more than 63 bits for long-alignment. https://github.com/apache/solr/pull/2248/commits/b1ac97649081aed058

Re: [PR] SOLR-13350: Multithreaded search [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on code in PR #2248: URL: https://github.com/apache/solr/pull/2248#discussion_r1565666020 ## solr/core/src/java/org/apache/solr/search/MultiThreadedSearcher.java: ## @@ -0,0 +1,415 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more

Re: [PR] SOLR-13350: Multithreaded search [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on code in PR #2248: URL: https://github.com/apache/solr/pull/2248#discussion_r1565662583 ## solr/core/src/java/org/apache/solr/search/MultiThreadedSearcher.java: ## @@ -0,0 +1,415 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-04-15 Thread via GitHub
cpoerschke commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1565648742 ## solr/modules/monitor/src/test-files/solr/collection1/solrconfig-no-cache.xml: ## @@ -0,0 +1,185 @@ + + + + + 9.4 Review Comment: minor/maybe ```suggestion