Re: [PR] SOLR-17275: Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases [solr]

2024-05-15 Thread via GitHub
dsmiley commented on PR #2463: URL: https://github.com/apache/solr/pull/2463#issuecomment-2113480275 Can you please update CHANGES.txt under 9.6.1 bug fix, maybe say: > SOLR-17275: SolrJ ZkClientClusterStateProvider, revert SOLR-17153 for regression when aliases are used. (your name) -

[jira] [Commented] (SOLR-17275) Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases

2024-05-15 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846758#comment-17846758 ] David Smiley commented on SOLR-17275: - I'll merge the change tomorrow morning to ensu

Re: [PR] SOLR-10654: Introduce output of Prometheus metrics for Solr Core registry [solr]

2024-05-15 Thread via GitHub
mlbiscoc commented on PR #2405: URL: https://github.com/apache/solr/pull/2405#issuecomment-2113445818 @dsmiley or anyone else interested in reviewing. Was curious for more feedback on approach. Is this going in the correct direction or do some disagree if this is correct implementation path

[jira] [Comment Edited] (SOLR-17275) Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases

2024-05-15 Thread Aparna Suresh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846729#comment-17846729 ] Aparna Suresh edited comment on SOLR-17275 at 5/15/24 6:53 PM:

Re: [PR] SOLR-17275: Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases [solr]

2024-05-15 Thread via GitHub
dsmiley commented on PR #2463: URL: https://github.com/apache/solr/pull/2463#issuecomment-2113240161 +1 for reverting this piece. I think we'll come back to it later when we have more time to reconsider it. I strongly suspected the root cause was an extra ZK interaction but your testing &

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
kotman12 commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1602087557 ## solr/modules/monitor/src/java/org/apache/solr/monitor/update/MonitorUpdateRequestProcessor.java: ## @@ -0,0 +1,257 @@ +/* + * + * * Licensed to the Apache Software Fo

[jira] [Commented] (SOLR-17275) Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases

2024-05-15 Thread Aparna Suresh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846729#comment-17846729 ] Aparna Suresh commented on SOLR-17275: -- I have reverted the change on the client sid

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
kotman12 commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1602065616 ## solr/modules/monitor/src/java/org/apache/lucene/monitor/MonitorFields.java: ## @@ -0,0 +1,38 @@ +/* + * + * * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
kotman12 commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1602065616 ## solr/modules/monitor/src/java/org/apache/lucene/monitor/MonitorFields.java: ## @@ -0,0 +1,38 @@ +/* + * + * * Licensed to the Apache Software Foundation (ASF) under o

[PR] SOLR-17275: Major performance regression of CloudSolrClient in Solr 9.6.0 when using aliases [solr]

2024-05-15 Thread via GitHub
aparnasuresh85 opened a new pull request, #2463: URL: https://github.com/apache/solr/pull/2463 https://issues.apache.org/jira/browse/SOLR-17275 # Description A recent change, as part of SOLR-17153 to ZkClusterStateProvider caused CloudSolrClient (zk based) to force

Re: [PR] SOLR-17106: LBSolrClient - Make zombie ping checks configurable [solr]

2024-05-15 Thread via GitHub
dsmiley commented on code in PR #2160: URL: https://github.com/apache/solr/pull/2160#discussion_r1601881299 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java: ## @@ -480,35 +488,70 @@ public RequestWriter getRequestWriter() { return requestWriter;

[jira] [Commented] (SOLR-14673) Add CLI for Streaming Expressions

2024-05-15 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14673?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846679#comment-17846679 ] Eric Pugh commented on SOLR-14673: -- I am poking around at the patch file to see if I can

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
cpoerschke commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1601740654 ## solr/modules/monitor/src/java/org/apache/lucene/monitor/MonitorFields.java: ## @@ -0,0 +1,38 @@ +/* + * + * * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] SOLR-17106: LBSolrClient - Make zombie ping checks configurable [solr]

2024-05-15 Thread via GitHub
epugh commented on code in PR #2160: URL: https://github.com/apache/solr/pull/2160#discussion_r1601674762 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttp2SolrClient.java: ## @@ -281,6 +298,37 @@ public LBHttp2SolrClient.Builder setAliveCheckInterval(int aliveChe

[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc

2024-05-15 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846638#comment-17846638 ] David Smiley commented on SOLR-17263: - I'm a bit confused by all these PRs but I merg

Re: [PR] SOLR-17106: LBSolrClient - Make zombie ping checks configurable [solr]

2024-05-15 Thread via GitHub
dsmiley commented on PR #2160: URL: https://github.com/apache/solr/pull/2160#issuecomment-2112567149 Aparna, are we just missing use of EnvUtils here? I know we've taken this change as-is (in our fork) but even we don't have that, thus it's hard to actually _use_ the changes configured her

[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc

2024-05-15 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846634#comment-17846634 ] ASF subversion and git services commented on SOLR-17263: Commit a

Re: [PR] SOLR-17109: Give security manager explicit read access to sharedLib [solr]

2024-05-15 Thread via GitHub
epugh commented on PR #2172: URL: https://github.com/apache/solr/pull/2172#issuecomment-2112502657 > I'm fine with merging this. @janhoy's comment is valid though, it would cause issues if `solr.sharedLib` has more than one directory do you want me to do it? Or I can leave it up to

[PR] SOLR-16505: Add auth listener to recovery client [solr]

2024-05-15 Thread via GitHub
iamsanjay opened a new pull request, #2462: URL: https://github.com/apache/solr/pull/2462 https://issues.apache.org/jira/browse/SOLR-16505 As previously noticed that whenever we re-create Solr client using existing `Http2SolrClient` then no listeners were passed to the new `Ht

Re: [PR] SOLR-17109: Give security manager explicit read access to sharedLib [solr]

2024-05-15 Thread via GitHub
tflobbe commented on PR #2172: URL: https://github.com/apache/solr/pull/2172#issuecomment-2112460299 I'm fine with merging this. @janhoy's comment is valid though, it would cause issues if `solr.sharedLib` has more than one directory -- This is an automated message from the Apache Git Ser

[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc

2024-05-15 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846620#comment-17846620 ] ASF subversion and git services commented on SOLR-17263: Commit b

[jira] [Commented] (SOLR-17263) HttpJdkSolrClient doesn't encode curly braces etc

2024-05-15 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17846619#comment-17846619 ] ASF subversion and git services commented on SOLR-17263: Commit 4

Re: [PR] SOLR-17263: HttpJdkSolrClient doesn't encode curly braces etc [solr]

2024-05-15 Thread via GitHub
dsmiley merged PR #2433: URL: https://github.com/apache/solr/pull/2433 -- 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

Re: [PR] SOLR-17109: Give security manager explicit read access to sharedLib [solr]

2024-05-15 Thread via GitHub
epugh commented on PR #2172: URL: https://github.com/apache/solr/pull/2172#issuecomment-2112425839 Also, @dsmiley looks like you approved this change, can we go ahead and get it merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] SOLR-17109: Give security manager explicit read access to sharedLib [solr]

2024-05-15 Thread via GitHub
epugh commented on PR #2172: URL: https://github.com/apache/solr/pull/2172#issuecomment-2112421629 Can someone explain the future of security manager in Java to me? My understanding is that it is going away (and no replacement??)... So, wouldn't it be better just to deprecate the securi

[jira] [Reopened] (SOLR-16505) Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2

2024-05-15 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Smiley reopened SOLR-16505: - Assignee: David Smiley > Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2 > -

Re: [PR] SOLR-17290: Update SyncStrategy and PeerSyncWithLeader to use the recovery Http2SolrClient [solr]

2024-05-15 Thread via GitHub
epugh commented on PR #2460: URL: https://github.com/apache/solr/pull/2460#issuecomment-2112418592 Would love some thoughts on how we could handle the auth scenarios better without having to write an explicit test for it on every piece of code, once with auth and once with out ;-) -- Thi

Re: [PR] SOLR-17290: Update SyncStrategy and PeerSyncWithLeader to use the recovery Http2SolrClient [solr]

2024-05-15 Thread via GitHub
iamsanjay commented on PR #2460: URL: https://github.com/apache/solr/pull/2460#issuecomment-2112288509 > +1; will merge tomorrow. > > > The solr client is being used for REQUESTRECOVERY and did not seem to need the auth for that. > > Maybe you know something I don't here; what

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
cpoerschke commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1601387931 ## solr/modules/monitor/src/java/org/apache/solr/monitor/update/MonitorUpdateRequestProcessor.java: ## @@ -0,0 +1,257 @@ +/* + * + * * Licensed to the Apache Software

Re: [PR] SOLR-4587: integrate lucene-monitor into solr [solr]

2024-05-15 Thread via GitHub
cpoerschke commented on code in PR #2382: URL: https://github.com/apache/solr/pull/2382#discussion_r1601379272 ## solr/modules/monitor/src/java/org/apache/solr/monitor/update/MonitorUpdateRequestProcessor.java: ## @@ -0,0 +1,257 @@ +/* + * + * * Licensed to the Apache Software

[I] Solr 8.11 with SolrMetrics produces duplicate samples with prometheus v2.52 [solr-operator]

2024-05-15 Thread via GitHub
perosb opened a new issue, #705: URL: https://github.com/apache/solr-operator/issues/705 Solr 8.11 seem to be producing duplicate metrics which is flooding the logs and triggering alerts. This is only when using latest prometheus v2.52.0. Related https://github.com/prometheus/prometh