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

2024-04-03 Thread via GitHub
jdyer1 merged PR #2374: URL: https://github.com/apache/solr/pull/2374 -- 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.org

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

2024-04-02 Thread via GitHub
jdyer1 commented on PR #2374: URL: https://github.com/apache/solr/pull/2374#issuecomment-2033116140 Unless there are objections I would like to merge this one soon. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

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

2024-04-02 Thread via GitHub
jdyer1 commented on code in PR #2374: URL: https://github.com/apache/solr/pull/2374#discussion_r1548589588 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpJdkSolrClient.java: ## @@ -469,6 +531,23 @@ protected String allProcessorSupportedContentTypesCommaDelimited(

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

2024-04-02 Thread via GitHub
jdyer1 commented on code in PR #2374: URL: https://github.com/apache/solr/pull/2374#discussion_r1548587323 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/HttpSolrClientTestBase.java: ## @@ -532,4 +538,67 @@ protected void testUseOptionalCredentialsWithNull(HttpSolrCli

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

2024-04-01 Thread via GitHub
dsmiley commented on code in PR #2374: URL: https://github.com/apache/solr/pull/2374#discussion_r1546911504 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpJdkSolrClient.java: ## @@ -469,6 +531,23 @@ protected String allProcessorSupportedContentTypesCommaDelimited(

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

2024-04-01 Thread via GitHub
jdyer1 commented on code in PR #2374: URL: https://github.com/apache/solr/pull/2374#discussion_r1546681105 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpJdkSolrClient.java: ## @@ -469,6 +531,23 @@ protected String allProcessorSupportedContentTypesCommaDelimited(

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, HttpServletResponse

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

2024-03-25 Thread via GitHub
jdyer1 opened a new pull request, #2374: URL: https://github.com/apache/solr/pull/2374 This PR adds method `asyncRequest` to abstract supertype `HttpSolrClientBase` and implements it on `HttpJdkSolrClient`. Also adds unit tests for normal, exceptional and cancelled asynchronous requests,