[GitHub] [solr] psalagnac commented on pull request #1465: SOLR-16705: Remove useless Zookeeper checks

2023-03-28 Thread via GitHub
psalagnac commented on PR #1465: URL: https://github.com/apache/solr/pull/1465#issuecomment-1486416452 Hi @markrmiller, Thanks for looking into this. Is this change good to be merged? Is something else required? Thanks -- This is an automated message from the Apache Git Service. T

[jira] [Commented] (SOLR-16465) Start the migration of the Admin UI to React

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17705878#comment-17705878 ] Jan Høydahl commented on SOLR-16465: FYI: https://lists.apache.org/thread/nfm6jj41bop

[jira] [Commented] (SOLR-16601) Remove Deprecated Methods from Solr Clients

2023-03-28 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17705935#comment-17705935 ] Eric Pugh commented on SOLR-16601: -- This is ready for review! Can removing the depreca

[jira] [Assigned] (SOLR-16601) Remove Deprecated Methods from Solr Clients

2023-03-28 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16601?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Pugh reassigned SOLR-16601: Assignee: Eric Pugh > Remove Deprecated Methods from Solr Clients > --

[GitHub] [solr] risdenk opened a new pull request, #1498: Cleanup commons lang3

2023-03-28 Thread via GitHub
risdenk opened a new pull request, #1498: URL: https://github.com/apache/solr/pull/1498 https://issues.apache.org/jira/browse/SOLR-X This is NOT ready for review yet. This just works down the list of commons lang3 usages and tries to replace them with JDK methods where possible.

[GitHub] [solr] epugh opened a new pull request, #1499: SOLR-16604: Use the Builders directly in unit tests

2023-03-28 Thread via GitHub
epugh opened a new pull request, #1499: URL: https://github.com/apache/solr/pull/1499 https://issues.apache.org/jira/browse/SOLR-16604 # Description Swap to using Builder directly. # Solution Using the non deprecated version of the Builders where possible,

[GitHub] [solr] epugh commented on pull request #1499: SOLR-16604: Use the Builders directly in unit tests

2023-03-28 Thread via GitHub
epugh commented on PR #1499: URL: https://github.com/apache/solr/pull/1499#issuecomment-1486887898 @dsmiley starting the migration ;-) -- 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

[GitHub] [solr-operator] HoustonPutman commented on pull request #343: Use global.imagePullSecrets at solr-operator helm chart #338

2023-03-28 Thread via GitHub
HoustonPutman commented on PR #343: URL: https://github.com/apache/solr-operator/pull/343#issuecomment-1487014768 @piotrrybicki Please open your own PR. We'd love to support as many globals as possible, to make setup easy for everyone! As for my comment above it would be great if the

[GitHub] [solr-operator] HoustonPutman commented on issue #534: zk provided pvc

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #534: URL: https://github.com/apache/solr-operator/issues/534#issuecomment-1487027991 Ahh so it looks like the zookeeper operator does support additional volumes, but the solr operator doesn't allow that functionality. We should make a PR to add the addition

[GitHub] [solr-operator] HoustonPutman commented on issue #533: Default liveliness and readiness probes return 401

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #533: URL: https://github.com/apache/solr-operator/issues/533#issuecomment-1487041283 Under `authentication` I think you need to add `"blockUnknown": false`. You are authorized for those endpoints, but since you are not providing the basic auth header, you a

[GitHub] [solr-operator] HoustonPutman opened a new issue, #535: Allow sessionAffinity for the SolrCloud CommonService

2023-03-28 Thread via GitHub
HoustonPutman opened a new issue, #535: URL: https://github.com/apache/solr-operator/issues/535 Kubernetes ClusterIP Services allow for sessionAffinity, and customization of the max session stickiness timeout. More information can be found here: https://kubernetes.io/docs/reference/n

[GitHub] [solr-operator] GiuseppeCSI commented on issue #533: Default liveliness and readiness probes return 401

2023-03-28 Thread via GitHub
GiuseppeCSI commented on issue #533: URL: https://github.com/apache/solr-operator/issues/533#issuecomment-1487088565 I think i misinterpreted this part > First, the probesRequireAuth setting (defaults to false) governs the value for blockUnknown (under authentication) and whether the

[GitHub] [solr] solrbot closed pull request #1492: Update dependency io.swagger.core.v3:swagger-annotations to v2.2.9 - autoclosed

2023-03-28 Thread via GitHub
solrbot closed pull request #1492: Update dependency io.swagger.core.v3:swagger-annotations to v2.2.9 - autoclosed URL: https://github.com/apache/solr/pull/1492 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[jira] [Commented] (SOLR-16601) Remove Deprecated Methods from Solr Clients

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706061#comment-17706061 ] Jan Høydahl commented on SOLR-16601: All public facing SolrJ code deprecated after 9.

[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #1498: Cleanup commons lang3

2023-03-28 Thread via GitHub
sonatype-lift[bot] commented on code in PR #1498: URL: https://github.com/apache/solr/pull/1498#discussion_r1150808170 ## solr/core/src/java/org/apache/solr/cloud/ExclusiveSliceProperty.java: ## @@ -76,13 +77,13 @@ class ExclusiveSliceProperty { ExclusiveSliceProperty(Cluster

[GitHub] [solr-operator] HoustonPutman commented on issue #529: Support disruption free rolling restart

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #529: URL: https://github.com/apache/solr-operator/issues/529#issuecomment-1487136250 @janhoy we should also create a Solr JIRA issue for this, to fix Cloud-aware clients and internal shard requests. More info: We can fix this for simple use cases wher

[GitHub] [solr] janhoy commented on a diff in pull request #1496: SOLR-16601: Remove Deprecated Methods from SolrClients

2023-03-28 Thread via GitHub
janhoy commented on code in PR #1496: URL: https://github.com/apache/solr/pull/1496#discussion_r1150844208 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/ConcurrentUpdateSolrClient.java: ## @@ -116,12 +114,23 @@ protected ConcurrentUpdateSolrClient(Builder builder) {

[GitHub] [solr-operator] HoustonPutman closed issue #532: Updating Solr Document Results Duplicate Versions

2023-03-28 Thread via GitHub
HoustonPutman closed issue #532: Updating Solr Document Results Duplicate Versions URL: https://github.com/apache/solr-operator/issues/532 -- 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 specifi

[GitHub] [solr-operator] HoustonPutman commented on issue #532: Updating Solr Document Results Duplicate Versions

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #532: URL: https://github.com/apache/solr-operator/issues/532#issuecomment-1487210778 I'd post this question to the [solr users list](https://solr.apache.org/community.html#mailing-lists-chat), since this question is purely about Solr and not about interacti

[GitHub] [solr-operator] HoustonPutman commented on issue #533: Default liveliness and readiness probes return 401

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #533: URL: https://github.com/apache/solr-operator/issues/533#issuecomment-1487217238 Did you try to update an existing cluster or create a new one? Also even if you deleted and recreated, you need to make sure that Zookeeper didn't use the same persis

[GitHub] [solr-operator] janhoy commented on a diff in pull request #530: Add readinessCondition to stop traffic to pods that will be stopped

2023-03-28 Thread via GitHub
janhoy commented on code in PR #530: URL: https://github.com/apache/solr-operator/pull/530#discussion_r1150958026 ## controllers/util/solr_update_util_test.go: ## @@ -180,6 +205,26 @@ func TestPickPodsToUpgrade(t *testing.T) { solrCloud.Spec.Replicas = Replicas(4)

[GitHub] [solr-operator] janhoy commented on issue #529: Support disruption free rolling restart

2023-03-28 Thread via GitHub
janhoy commented on issue #529: URL: https://github.com/apache/solr-operator/issues/529#issuecomment-1487367447 > @janhoy we should also create a Solr JIRA issue for this, to fix Cloud-aware clients and internal shard requests. Sure, I can create one. Do you have a clear idea of how

[GitHub] [solr] risdenk opened a new pull request, #1500: remove-collections4

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

[GitHub] [solr] janhoy commented on a diff in pull request #1500: remove-collections4

2023-03-28 Thread via GitHub
janhoy commented on code in PR #1500: URL: https://github.com/apache/solr/pull/1500#discussion_r115011 ## solr/core/src/java/org/apache/solr/search/SolrDocumentFetcher.java: ## @@ -762,7 +761,7 @@ private boolean returnStoredFields() { } private boolean returnDVF

[GitHub] [solr] janhoy merged pull request #1489: Remove duplicate code fragments in StreamingTest

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

[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #1500: remove-collections4

2023-03-28 Thread via GitHub
sonatype-lift[bot] commented on code in PR #1500: URL: https://github.com/apache/solr/pull/1500#discussion_r1151031492 ## solr/core/src/java/org/apache/solr/packagemanager/PackageManager.java: ## @@ -307,15 +303,14 @@ public Map getPackagesDeployedAsClusterLevelPlugins i

[GitHub] [solr-operator] HoustonPutman commented on issue #529: Support disruption free rolling restart

2023-03-28 Thread via GitHub
HoustonPutman commented on issue #529: URL: https://github.com/apache/solr-operator/issues/529#issuecomment-1487463171 Not a clear idea yet. > Would we need some new per-node-state znode in Zookeeper to flag a node as "draining", and then let SolrJ act on that? That would work,

[GitHub] [solr] risdenk opened a new pull request, #1501: More guava cleanup

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

[jira] [Commented] (SOLR-16668) Use default Java SslContextFactory for HTTP2 when no system properties are given

2023-03-28 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706116#comment-17706116 ] Shawn Heisey commented on SOLR-16668: - {quote}Huh, I thought one of our drivers for u

[GitHub] [solr-docker] HoustonPutman commented on pull request #14: SOLR-16523 Update the jattach version to 2.1

2023-03-28 Thread via GitHub
HoustonPutman commented on PR #14: URL: https://github.com/apache/solr-docker/pull/14#issuecomment-1487473448 Ok so Solr 9.2 officially has jattach 2.0 built in: ``` ❯ docker run -it --rm solr:9.2 jattach --version jattach 2.0 built on Aug 16 2021 ``` So we can put jatt

[jira] [Commented] (SOLR-16668) Use default Java SslContextFactory for HTTP2 when no system properties are given

2023-03-28 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16668?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706118#comment-17706118 ] Shawn Heisey commented on SOLR-16668: - {quote}you might also need to upgrade Spring b

[GitHub] [solr] janhoy opened a new pull request, #1502: SOLR-16721 Java version detection fails when `_JAVA_OPTIONS` is set

2023-03-28 Thread via GitHub
janhoy opened a new pull request, #1502: URL: https://github.com/apache/solr/pull/1502 https://issues.apache.org/jira/browse/SOLR-16721 Simple fix. Removing the `head` filter. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[jira] [Assigned] (SOLR-16721) Java version detection fails when `_JAVA_OPTIONS` is set

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16721?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jan Høydahl reassigned SOLR-16721: -- Assignee: Jan Høydahl > Java version detection fails when `_JAVA_OPTIONS` is set > ---

[jira] [Commented] (SOLR-16721) Java version detection fails when `_JAVA_OPTIONS` is set

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706133#comment-17706133 ] Jan Høydahl commented on SOLR-16721: [~zhongruoyu] I reproduced the issue by setting

[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #1501: More guava cleanup

2023-03-28 Thread via GitHub
sonatype-lift[bot] commented on code in PR #1501: URL: https://github.com/apache/solr/pull/1501#discussion_r1151106048 ## solr/core/src/java/org/apache/solr/logging/jul/JulWatcher.java: ## @@ -157,7 +158,9 @@ public SolrDocument toSolrDocument(LogRecord event) { doc.setFiel

[GitHub] [solr] risdenk commented on a diff in pull request #1496: SOLR-16601: Remove Deprecated Methods from SolrClients

2023-03-28 Thread via GitHub
risdenk commented on code in PR #1496: URL: https://github.com/apache/solr/pull/1496#discussion_r1151109774 ## solr/core/src/test/org/apache/solr/cloud/TestRandomFlRTGCloud.java: ## @@ -147,6 +148,7 @@ public class TestRandomFlRTGCloud extends SolrCloudTestCase { new

[jira] [Created] (SOLR-16722) API to flag a solr node NOT READY for requests

2023-03-28 Thread Jira
Jan Høydahl created SOLR-16722: -- Summary: API to flag a solr node NOT READY for requests Key: SOLR-16722 URL: https://issues.apache.org/jira/browse/SOLR-16722 Project: Solr Issue Type: New Featu

[GitHub] [solr-operator] janhoy commented on issue #529: Support disruption free rolling restart

2023-03-28 Thread via GitHub
janhoy commented on issue #529: URL: https://github.com/apache/solr-operator/issues/529#issuecomment-1487551816 https://issues.apache.org/jira/browse/SOLR-16722 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[jira] [Commented] (SOLR-16722) API to flag a solr node NOT READY for requests

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706145#comment-17706145 ] Jan Høydahl commented on SOLR-16722: One simple option is to create a new top-level z

[GitHub] [solr] risdenk commented on a diff in pull request #1407: SOLR-16427: Evaluate and fix errorprone rules - part 5

2023-03-28 Thread via GitHub
risdenk commented on code in PR #1407: URL: https://github.com/apache/solr/pull/1407#discussion_r1151134327 ## solr/solrj/src/java/org/noggit/JSONUtil.java: ## @@ -17,11 +17,19 @@ package org.noggit; public class JSONUtil { + @SuppressWarnings("MutablePublicArray") publi

[jira] [Created] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
Kevin Risden created SOLR-16723: --- Summary: Http2SolrClient should not use Apache Http client classes Key: SOLR-16723 URL: https://issues.apache.org/jira/browse/SOLR-16723 Project: Solr Issue Ty

[jira] [Updated] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-16723: Description: https://lists.apache.org/thread/vz6gldbdq0gylncpwsoshfvlvns4kqv8 > Http2SolrClient

[jira] [Updated] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-16723: Affects Version/s: 9.2 > Http2SolrClient should not use Apache Http client classes > --

[jira] [Updated] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-16723: Description: @elyograg pointed out that Http2SolrClient has Apache Http Client class in use and it

[jira] [Updated] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-16723: Priority: Minor (was: Major) > Http2SolrClient should not use Apache Http client classes > ---

[GitHub] [solr] risdenk opened a new pull request, #1503: SOLR-16723: Http2SolrClient should not use Apache Http client classes

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

[jira] [Updated] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-16723: Status: Patch Available (was: Open) > Http2SolrClient should not use Apache Http client classes >

[jira] [Commented] (SOLR-16722) API to flag a solr node NOT READY for requests

2023-03-28 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706151#comment-17706151 ] Jan Høydahl commented on SOLR-16722: Another obvious option would be for the Solr Ope

[GitHub] [solr-operator] hmontesdeoca commented on issue #534: Ability to add additional volumes to Zookeeper Pods

2023-03-28 Thread via GitHub
hmontesdeoca commented on issue #534: URL: https://github.com/apache/solr-operator/issues/534#issuecomment-1487639416 thank you for the information, I will relay this to my team. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [solr] noblepaul commented on pull request #1485: SOLR-16507: Remove NodeStateProvider & Snitch

2023-03-28 Thread via GitHub
noblepaul commented on PR #1485: URL: https://github.com/apache/solr/pull/1485#issuecomment-1487748631 LGTM , `snitch` must be removed anyway -- 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 s

[jira] [Commented] (SOLR-16723) Http2SolrClient should not use Apache Http client classes

2023-03-28 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706204#comment-17706204 ] David Smiley commented on SOLR-16723: - SOLR-16079 -- once we modularize the HttpClien

[jira] [Commented] (SOLR-16465) Start the migration of the Admin UI to React

2023-03-28 Thread Marcus Eagan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17706228#comment-17706228 ] Marcus Eagan commented on SOLR-16465: - Thanks [~janhoy] that's helpful. [~gus] regar