Re: [PR] SOLR-17220 - Make the SolrZkClient thread as a daemon thread [solr]

2024-03-30 Thread via GitHub
dsmiley commented on PR #2376: URL: https://github.com/apache/solr/pull/2376#issuecomment-2028438789 +1 to murblanc -- SolrClient.close must be called -- 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

Re: [PR] SOLR-16677: Updated Solr to use new Lucene 9.5 storedFields() API [solr]

2024-03-30 Thread via GitHub
dsmiley commented on code in PR #1557: URL: https://github.com/apache/solr/pull/1557#discussion_r1545466879 ## solr/core/src/java/org/apache/solr/search/SolrDocumentFetcher.java: ## @@ -114,9 +114,33 @@ public class SolrDocumentFetcher { private final Set largeFields; -

Re: [PR] SOLR-16505: Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2 [solr]

2024-03-30 Thread via GitHub
dsmiley commented on code in PR #2276: URL: https://github.com/apache/solr/pull/2276#discussion_r1545301666 ## solr/core/src/java/org/apache/solr/handler/IndexFetcher.java: ## @@ -1828,13 +1823,17 @@ private int fetchPackets(FastInputStream fis) throws Exception {

Re: [PR] SOLR-16505: Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2 [solr]

2024-03-30 Thread via GitHub
dsmiley commented on code in PR #2276: URL: https://github.com/apache/solr/pull/2276#discussion_r1545293836 ## solr/core/src/java/org/apache/solr/handler/IndexFetcher.java: ## @@ -1828,13 +1823,17 @@ private int fetchPackets(FastInputStream fis) throws Exception {

Re: [PR] SOLR-17211: HttpJdkSolrClient Support Async requests [solr]

2024-03-30 Thread via GitHub
dsmiley commented on code in PR #2374: URL: https://github.com/apache/solr/pull/2374#discussion_r1545278812 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/DebugServlet.java: ## @@ -136,6 +137,16 @@ private void recordRequest(HttpServletRequest req,

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

2024-03-30 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14763?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17832434#comment-17832434 ] David Smiley commented on SOLR-14763: - This missed 9.0 sadly; I suppose we could add an adapter

[jira] [Commented] (SOLR-17211) HttpJdkSolrClient: Support Async

2024-03-30 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17832433#comment-17832433 ] David Smiley commented on SOLR-17211: - James, please see SOLR-14763 which suggests an API that

Re: [PR] SOLR-16505: Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2 [solr]

2024-03-30 Thread via GitHub
iamsanjay commented on PR #2276: URL: https://github.com/apache/solr/pull/2276#issuecomment-2027942595 Fix for RST_STREAM! Upon enabling logs at "org.eclipse.jetty" level. It was found that IndexFetcher was closing the InputStream prematurely, causing the client to send RST_FRAME