gerlowskija commented on PR #636:
URL: https://github.com/apache/solr-operator/pull/636#issuecomment-1744089624
> might as well include the solr.port.advertise here as well
I've added the necessary changes to work `solr.port.advertise` into
generated solr.xml files, though I think it'
[
https://issues.apache.org/jira/browse/SOLR-17000?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771309#comment-17771309
]
Kevin Risden commented on SOLR-17000:
-
[^SOLR-17000-1.patch] looks good
> ExecutorUt
[
https://issues.apache.org/jira/browse/SOLR-17000?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chris M. Hostetter updated SOLR-17000:
--
Attachment: SOLR-17000-1.patch
Status: Open (was: Open)
Updated patch to use
gerlowskija commented on PR #636:
URL: https://github.com/apache/solr-operator/pull/636#issuecomment-1744002994
Ah, yes - sorry, I was being dense. Had an epiphany about what you meant
while I was afk getting dinner. I tested it as well - looks good on 8.11 and
9.3 both. So I think we're
risdenk merged PR #1968:
URL: https://github.com/apache/solr/pull/1968
--
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.or
risdenk merged PR #1815:
URL: https://github.com/apache/solr/pull/1815
--
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.or
risdenk merged PR #1970:
URL: https://github.com/apache/solr/pull/1970
--
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.or
risdenk merged PR #1966:
URL: https://github.com/apache/solr/pull/1966
--
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.or
risdenk merged PR #1953:
URL: https://github.com/apache/solr/pull/1953
--
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.or
markrmiller commented on PR #73:
URL: https://github.com/apache/solr-sandbox/pull/73#issuecomment-1743774649
Good catch, big hole that that test only tests dropping zk and not Solr.
I'm just wrapping up a fix.
Appears to be caused by batching multiple Kafka records to a SolrRequ
HoustonPutman commented on PR #636:
URL: https://github.com/apache/solr-operator/pull/636#issuecomment-1743747944
> > If the user doesn't provide solr.sharedLib, the xml section will start
with ,.
>
> Hmm, not quite sure what you mean. Are you talking about the
[GenerateAdditionalLib
gerlowskija commented on PR #636:
URL: https://github.com/apache/solr-operator/pull/636#issuecomment-1743725754
> If the user doesn't provide solr.sharedLib, the xml section will start
with ,.
Hmm, not quite sure what you mean. Are you talking about the
[GenerateAdditionalLibXMLPart
HoustonPutman commented on PR #639:
URL: https://github.com/apache/solr-operator/pull/639#issuecomment-1743716056
> 1. How do we want to handle the potential conflict between user-specified
env-vars, and values like `.Spec.NumThreads` that are converted into an env-var
by the operator? Cur
gerlowskija commented on PR #639:
URL: https://github.com/apache/solr-operator/pull/639#issuecomment-1743692444
Two questions I could use a bit of clarity on before this can be merged:
1. How do we want to handle the potential conflict between user-specified
env-vars, and values like
[
https://issues.apache.org/jira/browse/SOLR-14496?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Eric Pugh updated SOLR-14496:
-
Description:
This ticket will allow all the logical tools in bin/solr (SolrCLI) to use basic
auth creden
gerlowskija opened a new pull request, #639:
URL: https://github.com/apache/solr-operator/pull/639
In recent versions of Solr (>=8.8) the prometheus exporter has an env-var
equivalent for each CLI argument taken by its start script.
This commit switches the operator over to providing
gerlowskija opened a new issue, #638:
URL: https://github.com/apache/solr-operator/issues/638
In recent versions of Solr (>=8.8) the prometheus exporter has an env-var
equivalent for each CLI argument taken in by its start script.
Env vars can be easier to read and parse than a long s
risdenk commented on PR #1967:
URL: https://github.com/apache/solr/pull/1967#issuecomment-1743661371
Should wait for 4.1.99.Final to end up on this PR.
--
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 g
epugh commented on PR #1954:
URL: https://github.com/apache/solr/pull/1954#issuecomment-1743656420
Still need to update the Ref Guide... Hoping to get clarity on if we can
re-purpose the `-u` parameter ;-)
--
This is an automated message from the Apache Git Service.
To respond to the mes
epugh commented on PR #1954:
URL: https://github.com/apache/solr/pull/1954#issuecomment-1743655383
> `Dbasicauth`
I looked at `test_ssl.bats` and it has these lines:
```
solr start -c
solr auth enable -type basicAuth -credentials name:password
solr assert --starte
risdenk merged PR #1812:
URL: https://github.com/apache/solr/pull/1812
--
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.or
risdenk merged PR #1877:
URL: https://github.com/apache/solr/pull/1877
--
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.or
risdenk merged PR #1608:
URL: https://github.com/apache/solr/pull/1608
--
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.or
AndreyBozhko commented on PR #1961:
URL: https://github.com/apache/solr/pull/1961#issuecomment-1743605262
@epugh This should be ready now for your review.
--
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 t
risdenk merged PR #1910:
URL: https://github.com/apache/solr/pull/1910
--
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.or
epugh commented on PR #1954:
URL: https://github.com/apache/solr/pull/1954#issuecomment-1743526141
> Will this still support using `-Dbasicauth` and
`-Dsolr.httpclient.builder.factory` in `SOLR_TOOL_OPTS`?
It *should*, however, to be honest, I've never used those parameters...
Do
HoustonPutman commented on PR #1954:
URL: https://github.com/apache/solr/pull/1954#issuecomment-1743507341
Will this still support using `-Dbasicauth` and
`-Dsolr.httpclient.builder.factory` in `SOLR_TOOL_OPTS`?
--
This is an automated message from the Apache Git Service.
To respond to th
[
https://issues.apache.org/jira/browse/SOLR-16960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771235#comment-17771235
]
ASF subversion and git services commented on SOLR-16960:
Commit b
stillalex merged PR #1972:
URL: https://github.com/apache/solr/pull/1972
--
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.
HoustonPutman commented on PR #636:
URL: https://github.com/apache/solr-operator/pull/636#issuecomment-1743489917
Oh, might as well include the `solr.port.advertise` here as well, unless you
wanted to do that in a separate PR. No worries either way.
--
This is an automated message from th
stillalex opened a new pull request, #1972:
URL: https://github.com/apache/solr/pull/1972
https://issues.apache.org/jira/browse/SOLR-16960
# Description
Regression of SOLR-16960 in the case where the recoding noop span can
override the simple trace id with a default
risdenk merged PR #1701:
URL: https://github.com/apache/solr/pull/1701
--
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.or
solrbot commented on PR #1701:
URL: https://github.com/apache/solr/pull/1701#issuecomment-1743327720
### Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not
recognize the last commit author and assumes somebody else may have edited the
PR.
stillalex commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342889198
##
solr/core/src/java/org/apache/solr/cli/AssertTool.java:
##
@@ -123,6 +123,14 @@ public List getOptions() {
Option.builder("e")
.desc("Return an
Ella-eln opened a new issue, #637:
URL: https://github.com/apache/solr-operator/issues/637
Pravega's zk operator only allows to expose adminserverservice as external
out of the box. Solr itself does not have a problem with it, however, it
creates a problem when trying to write data into sol
epugh commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342815745
##
solr/core/src/java/org/apache/solr/cli/ExportTool.java:
##
@@ -202,7 +209,16 @@ DocsSink getSink() {
abstract void exportDocs() throws Exception;
void fetchUn
[
https://issues.apache.org/jira/browse/SOLR-16843?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Pierre Salagnac updated SOLR-16843:
---
Description:
The framework does not exist anymore with Solr 9+, so this bug only impacts
Sol
epugh commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342810570
##
solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java:
##
@@ -1311,6 +1311,22 @@ public Builder withProxyConfiguration(
this.proxyIsSecure = i
epugh commented on PR #1954:
URL: https://github.com/apache/solr/pull/1954#issuecomment-1743178358
So, one thing I noticed is that the method
`withOptionalBasicAuthCredentials` is ONLY on Http2SolrClient.Builder. It
isn't on the `HttpSolrClient builder`. Or on some of the other variations
risdenk commented on PR #1701:
URL: https://github.com/apache/solr/pull/1701#issuecomment-1743173285
Hmmm well apparently locally didn't end up recompiling all the tests so
doing that now :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
epugh commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342785865
##
solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java:
##
@@ -1311,6 +1311,22 @@ public Builder withProxyConfiguration(
this.proxyIsSecure = i
risdenk commented on PR #1701:
URL: https://github.com/apache/solr/pull/1701#issuecomment-1743101457
I'm looking at this today
--
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
[
https://issues.apache.org/jira/browse/SOLR-17004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771133#comment-17771133
]
ASF subversion and git services commented on SOLR-17004:
Commit 3
[
https://issues.apache.org/jira/browse/SOLR-17004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kevin Risden updated SOLR-17004:
Resolution: Fixed
Status: Resolved (was: Patch Available)
> ZkStateReader waitForState sho
[
https://issues.apache.org/jira/browse/SOLR-17004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kevin Risden updated SOLR-17004:
Fix Version/s: 9.4
> ZkStateReader waitForState should check clusterState before using watchers
> -
[
https://issues.apache.org/jira/browse/SOLR-17004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771130#comment-17771130
]
ASF subversion and git services commented on SOLR-17004:
Commit 6
epugh commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342625748
##
solr/core/src/java/org/apache/solr/cli/AssertTool.java:
##
@@ -123,6 +123,14 @@ public List getOptions() {
Option.builder("e")
.desc("Return an exit
epugh commented on code in PR #1954:
URL: https://github.com/apache/solr/pull/1954#discussion_r1342625290
##
solr/core/src/java/org/apache/solr/cli/ApiTool.java:
##
@@ -57,27 +57,28 @@ public List getOptions() {
.hasArg()
.required(true)
.d
dsmiley commented on PR #1962:
URL: https://github.com/apache/solr/pull/1962#issuecomment-1742899303
Fantastic docs on CoordinatorHttpSolrCall :-)
--
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
marcingorski commented on PR #70:
URL: https://github.com/apache/solr-sandbox/pull/70#issuecomment-1742859381
If topicName/dlqTopicName is specified but not available on Kafka server, we
will throw exception.
Log level has been changed to `INFO`.
--
This is an automated message from t
[
https://issues.apache.org/jira/browse/SOLR-16994?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Christine Poerschke resolved SOLR-16994.
Fix Version/s: main (10.0)
9.4
Resolution: Fixed
> facili
[
https://issues.apache.org/jira/browse/SOLR-16994?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771042#comment-17771042
]
ASF subversion and git services commented on SOLR-16994:
Commit 2
[
https://issues.apache.org/jira/browse/SOLR-16994?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771040#comment-17771040
]
ASF subversion and git services commented on SOLR-16994:
Commit e
cpoerschke merged PR #1950:
URL: https://github.com/apache/solr/pull/1950
--
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
54 matches
Mail list logo