[jira] [Created] (SOLR-17456) TransactionLog NPE

2024-09-16 Thread David Smiley (Jira)
David Smiley created SOLR-17456: --- Summary: TransactionLog NPE Key: SOLR-17456 URL: https://issues.apache.org/jira/browse/SOLR-17456 Project: Solr Issue Type: Bug Security Level: Public (D

[jira] [Commented] (SOLR-17256) Remove SolrRequest.getBasePath setBasePath

2024-09-16 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17882198#comment-17882198 ] David Smiley commented on SOLR-17256: - Another approach would be for Http2SolrClient.

Re: [PR] SOLR-17256: Replace SolrRequest.setBasePath (PoC) [solr]

2024-09-16 Thread via GitHub
dsmiley commented on code in PR #2714: URL: https://github.com/apache/solr/pull/2714#discussion_r1761986518 ## solr/solrj/src/java/org/apache/solr/client/solrj/util/ClientUtils.java: ## @@ -102,6 +111,24 @@ public static String buildRequestUrl( return basePath + path; }

Re: [PR] Feature/solr-17334 Minor bugs in Solr dedicated coordinator mode [solr]

2024-09-16 Thread via GitHub
ellaeln commented on PR #2672: URL: https://github.com/apache/solr/pull/2672#issuecomment-2353975200 > Hello. Next steps here could be to: > > 1. review and address the 'gradle precommit' failure above > 2. review the suggestions (and apply them if they are agreeable) I inves

Re: [PR] SOLR-14673: Add bin/solr stream CLI [solr]

2024-09-16 Thread via GitHub
gerlowskija commented on code in PR #2479: URL: https://github.com/apache/solr/pull/2479#discussion_r1761633393 ## solr/solr-ref-guide/modules/query-guide/pages/stream-tool.adoc: ## @@ -0,0 +1,167 @@ += Stream Tool +// Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] SOLR-14673: Add bin/solr stream CLI [solr]

2024-09-16 Thread via GitHub
gerlowskija commented on code in PR #2479: URL: https://github.com/apache/solr/pull/2479#discussion_r1760947380 ## solr/solr-ref-guide/modules/query-guide/pages/stream-tool.adoc: ## @@ -0,0 +1,167 @@ += Stream Tool +// Licensed to the Apache Software Foundation (ASF) under one +

[PR] Blog post - Community Over Code NA 2024 [solr-site]

2024-09-16 Thread via GitHub
lizbiella opened a new pull request, #126: URL: https://github.com/apache/solr-site/pull/126 A new blog post about the Community Over Code NA 2024. Let me know if publishing blog news about ASF (and Apache Solr) events and conferences is ok. -- This is an automated message from the Apache

[jira] [Comment Edited] (SOLR-17256) Remove SolrRequest.getBasePath setBasePath

2024-09-16 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17882039#comment-17882039 ] Jason Gerlowski edited comment on SOLR-17256 at 9/16/24 2:41 PM: --

[jira] [Commented] (SOLR-17256) Remove SolrRequest.getBasePath setBasePath

2024-09-16 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17882077#comment-17882077 ] Jason Gerlowski commented on SOLR-17256: [Here's a little PoC PR |https://github.

[jira] [Updated] (SOLR-17256) Remove SolrRequest.getBasePath setBasePath

2024-09-16 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated SOLR-17256: -- Labels: newdev pull-request-available (was: newdev) > Remove SolrRequest.getBasePath setBasePa

[jira] [Created] (SOLR-17455) Docs say we have tracing based on OpenTracing

2024-09-16 Thread Gus Heck (Jira)
Gus Heck created SOLR-17455: --- Summary: Docs say we have tracing based on OpenTracing Key: SOLR-17455 URL: https://issues.apache.org/jira/browse/SOLR-17455 Project: Solr Issue Type: Bug Securi

[PR] SOLR-17256: Replace SolrRequest.setBasePath (PoC) [solr]

2024-09-16 Thread via GitHub
gerlowskija opened a new pull request, #2714: URL: https://github.com/apache/solr/pull/2714 https://issues.apache.org/jira/browse/SOLR-17256 # Description Mutating SolrRequest objects as setBasePath does is a bad practice, as the modification isn't documented and many users expect

[jira] [Commented] (SOLR-17256) Remove SolrRequest.getBasePath setBasePath

2024-09-16 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17882039#comment-17882039 ] Jason Gerlowski commented on SOLR-17256: bq. [David] I think there's a conceptua

Re: [PR] SOLR-17406: Introduce Version Catalogs [solr]

2024-09-16 Thread via GitHub
malliaridis commented on code in PR #2706: URL: https://github.com/apache/solr/pull/2706#discussion_r1761060353 ## help/dependencies.txt: ## @@ -29,60 +29,126 @@ testImplementation - makes a dependency only available for test classes. Adding a library dependency

Re: [PR] SOLR-17406: Introduce Version Catalogs [solr]

2024-09-16 Thread via GitHub
janhoy commented on code in PR #2706: URL: https://github.com/apache/solr/pull/2706#discussion_r1761009874 ## help/dependencies.txt: ## @@ -29,60 +29,126 @@ testImplementation - makes a dependency only available for test classes. Adding a library dependency -

Re: [PR] SOLR-17450 StatusTool with pure Java code [solr]

2024-09-16 Thread via GitHub
janhoy commented on PR #2712: URL: https://github.com/apache/solr/pull/2712#issuecomment-2352420107 So the reason is that in `BasicAuthIntegrationTest` it calls the tool directly with a url to query the status endpoint of the local Solr being tested. However, our test suite does not actuall

[jira] [Created] (SOLR-17454) ERROR message in logs with multithreaded searches

2024-09-16 Thread Andrew Hankinson (Jira)
Andrew Hankinson created SOLR-17454: --- Summary: ERROR message in logs with multithreaded searches Key: SOLR-17454 URL: https://issues.apache.org/jira/browse/SOLR-17454 Project: Solr Issue Ty

Re: [PR] SOLR-17450 StatusTool with pure Java code [solr]

2024-09-16 Thread via GitHub
janhoy commented on PR #2712: URL: https://github.com/apache/solr/pull/2712#issuecomment-2352314226 There's a reproducible failure since `BasicAuthIntegrationTest` actually uses StatusTool and expects it to output pure JSON with the `--solr-url` option, and for it to also handle basic auth