[jira] [Commented] (SOLR-16350) http response is delayed when solr is running in slow/restricted network environment

2022-11-02 Thread Shubanker (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628120#comment-17628120 ] Shubanker commented on SOLR-16350: -- [~krisden]  Yes I did try accessing it over 127.0.0.1, the delay is

[jira] [Commented] (SOLR-16414) Race condition in PRS state updates

2022-11-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628111#comment-17628111 ] ASF subversion and git services commented on SOLR-16414: Commit

[jira] [Commented] (SOLR-16414) Race condition in PRS state updates

2022-11-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628107#comment-17628107 ] ASF subversion and git services commented on SOLR-16414: Commit

[jira] [Commented] (SOLR-16414) Race condition in PRS state updates

2022-11-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628105#comment-17628105 ] ASF subversion and git services commented on SOLR-16414: Commit

[jira] [Reopened] (SOLR-16414) Race condition in PRS state updates

2022-11-02 Thread Ishan Chattopadhyaya (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16414?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ishan Chattopadhyaya reopened SOLR-16414: - The last commit caused a massive slowdown in node stop event. With 1000 cores/node,

[GitHub] [solr] noblepaul commented on a diff in pull request #1159: SOLR-16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
noblepaul commented on code in PR #1159: URL: https://github.com/apache/solr/pull/1159#discussion_r1012271453 ## solr/solrj/src/java/org/apache/solr/common/cloud/ClusterState.java: ## @@ -295,15 +296,12 @@ private static DocCollection collectionFromObjects( } @Override

[jira] [Commented] (SOLR-16490) Create a v2 equivalent for /admin/cores BACKUPCORE+RESTORECORE

2022-11-02 Thread Sanjay Dutt (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628021#comment-17628021 ] Sanjay Dutt commented on SOLR-16490: I am working on the v2 BACKUPCORE API. Will submit the PR in

[GitHub] [solr-operator] janhoy opened a new issue, #488: Support for updating security.json through Operator / Helm

2022-11-02 Thread GitBox
janhoy opened a new issue, #488: URL: https://github.com/apache/solr-operator/issues/488 A user can now bootstrap `security.json` through operator, even by providing the raw `security.json` file in a Secret that will be uploaded to ZK on first Solr Pod start through an init-container.

[GitHub] [solr-operator] janhoy commented on issue #85: Support uploading configsets through the operator

2022-11-02 Thread GitBox
janhoy commented on issue #85: URL: https://github.com/apache/solr-operator/issues/85#issuecomment-1301476051 First question is - should it be a configSet bootstrapping feature that will only be run on first startup if a configset with that name does not already exist... OR - should

[jira] [Commented] (SOLR-16468) Create v2 equivalent of v1 'CREATESNAPSHOT', 'LISTSNAPSHOT' and 'DELETESNAPSHOT' (collection level)

2022-11-02 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17628004#comment-17628004 ] Jason Gerlowski commented on SOLR-16468: I'm going to grab LISTSNAPSHOT and push up a PR shortly.

[jira] [Assigned] (SOLR-16468) Create v2 equivalent of v1 'CREATESNAPSHOT', 'LISTSNAPSHOT' and 'DELETESNAPSHOT' (collection level)

2022-11-02 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16468?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gerlowski reassigned SOLR-16468: -- Assignee: Jason Gerlowski > Create v2 equivalent of v1 'CREATESNAPSHOT',

[GitHub] [solr] dsmiley commented on a diff in pull request #862: SOLR-16195: Correct task-mgmt API paths in ref-guide

2022-11-02 Thread GitBox
dsmiley commented on code in PR #862: URL: https://github.com/apache/solr/pull/862#discussion_r1012271513 ## build.gradle: ## @@ -207,3 +207,5 @@ apply from: file('gradle/ant-compat/solr.post-jar.gradle') apply from: file('gradle/node.gradle') +sourceCompatibility =

[GitHub] [solr] noblepaul commented on a diff in pull request #1159: SOLR-16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
noblepaul commented on code in PR #1159: URL: https://github.com/apache/solr/pull/1159#discussion_r1012271453 ## solr/solrj/src/java/org/apache/solr/common/cloud/ClusterState.java: ## @@ -295,15 +296,12 @@ private static DocCollection collectionFromObjects( } @Override

[GitHub] [solr-operator] jkgonzio opened a new issue, #487: CVE Security Vulnerabilities

2022-11-02 Thread GitBox
jkgonzio opened a new issue, #487: URL: https://github.com/apache/solr-operator/issues/487 I ran a flexline scan and ran into some security vulnerabilities that should be remediated: Vulnerability ID: CVE-2021-33194 Finding Level: High Package: golang.org/x/net/html Version:

[jira] [Resolved] (SOLR-16517) Solr export 8.11.2-slim Vulnerabilities

2022-11-02 Thread Houston Putman (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16517?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Houston Putman resolved SOLR-16517. --- Resolution: Invalid Please read our security page: https://solr.apache.org/security.html

[jira] [Updated] (SOLR-16517) Solr export 8.11.2-slim Vulnerabilities

2022-11-02 Thread Ritchie Gu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16517?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ritchie Gu updated SOLR-16517: -- Description: I have a question regarding the solr exporter

[jira] [Created] (SOLR-16517) Solr export 8.11.2-slim Vulnerabilities

2022-11-02 Thread Ritchie Gu (Jira)
Ritchie Gu created SOLR-16517: - Summary: Solr export 8.11.2-slim Vulnerabilities Key: SOLR-16517 URL: https://issues.apache.org/jira/browse/SOLR-16517 Project: Solr Issue Type: Bug

[GitHub] [solr] dsmiley commented on pull request #1156: [SOLR-16515] Remove synchronized access to cachedOrdMaps in SlowCompositeReaderWrapper

2022-11-02 Thread GitBox
dsmiley commented on PR #1156: URL: https://github.com/apache/solr/pull/1156#issuecomment-1300871894 I don't mean change Lucene code, I mean change SlowCompositeReaderWrapper internals only. Yeah Multi-stuff is kind of discouraged... more of a -- hey, use this if your algorithm can't

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300831545 DisjunctionMaxQuery unfortunately has a getDisjuncts that returns a mutable Multiset (a Lucene collection subclass):

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300811132 This is a non-Lucene or non-Solr query from the external Querqy plugin.

[GitHub] [solr] ben-manes commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
ben-manes commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300803918 oh yes, I did not notice that `Collection getClauses(Occur occur)` is package-private. This is returning a mutable collection, but since it is not public then that is safe (maybe

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300772734 Yes this helps. Interestingly: The "clauses" of a BooleanQuery are non-mutable, it is already wrapped with Collections.unmodifiableList(). So nobody can change the clauses after calling

[GitHub] [solr-site] HoustonPutman merged pull request #81: Warn about scans in security section. (#80)

2022-11-02 Thread GitBox
HoustonPutman merged PR #81: URL: https://github.com/apache/solr-site/pull/81 -- 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:

[GitHub] [solr-site] HoustonPutman opened a new pull request, #81: Warn about scans in security section. (#80)

2022-11-02 Thread GitBox
HoustonPutman opened a new pull request, #81: URL: https://github.com/apache/solr-site/pull/81 Co-authored-by: David Smiley -- 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.

[jira] [Commented] (SOLR-16496) provide option for Query Elevation Component to bypass filters

2022-11-02 Thread Rudi Seitz (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627788#comment-17627788 ] Rudi Seitz commented on SOLR-16496: --- Thanks, [~dsmiley], I've updated the PR based on your feedback.

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011906493 ## solr/solrj/src/test/org/apache/solr/client/solrj/SolrExampleTests.java: ## @@ -602,58 +602,60 @@ public void testUnicode() throws Exception { SolrClient client =

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011904785 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/BasicHttpSolrClientTest.java: ## @@ -478,6 +484,8 @@ public void testUpdate() throws Exception {

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011902917 ## solr/solrj/src/test/org/apache/solr/client/solrj/impl/BasicHttpSolrClientTest.java: ## @@ -341,7 +346,6 @@ public void testQuery() throws Exception {

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011897088 ## solr/solrj/src/test/org/apache/solr/client/solrj/TestBatchUpdate.java: ## @@ -46,24 +46,28 @@ public static void beforeTest() throws Exception { @Test public

[GitHub] [solr] rseitz commented on pull request #1154: SOLR-16496: provide option for Query Elevation Component to bypass filters

2022-11-02 Thread GitBox
rseitz commented on PR #1154: URL: https://github.com/apache/solr/pull/1154#issuecomment-1300609618 Thanks for reviewing, @risdenk and thanks to @dsmiley for feedback on the JIRA ticket. I have reworked this PR in accordance with that feedback. -- This is an automated message from the

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011894050 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrClient.java: ## @@ -813,6 +813,7 @@ public String getBaseURL() { * In this case the client is more

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011891480 ## solr/core/src/test/org/apache/solr/search/TestSmileRequest.java: ## @@ -71,7 +71,7 @@ public void testDistribJsonRequest() throws Exception { @Override

[GitHub] [solr] DennisBerger1984 commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
DennisBerger1984 commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300604173 @uschindler does this help? ![1](https://user-images.githubusercontent.com/69144692/199523879-cdddb7e3-f23c-4c5d-9cde-7099f35dd870.png)

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011888554 ## solr/solrj/src/test/org/apache/solr/client/solrj/embedded/SolrExampleJettyTest.java: ## @@ -118,72 +111,22 @@ public void testUtf8PerfDegradation() throws Exception {

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011881388 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/ConcurrentUpdateSolrClient.java: ## @@ -107,8 +107,8 @@ protected ConcurrentUpdateSolrClient(Builder builder) {

[GitHub] [solr] risdenk commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
risdenk commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1300573313 https://github.com/apache/solr/search?q=setWrappedQuery is definitely not used in Solr main. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [solr] risdenk commented on a diff in pull request #1159: SOLR-16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
risdenk commented on code in PR #1159: URL: https://github.com/apache/solr/pull/1159#discussion_r1011865290 ## solr/solrj/src/java/org/apache/solr/common/cloud/ClusterState.java: ## @@ -295,15 +296,12 @@ private static DocCollection collectionFromObjects( } @Override -

[GitHub] [solr] risdenk commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
risdenk commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011860272 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpSolrClient.java: ## @@ -813,6 +813,7 @@ public String getBaseURL() { * In this case the client is more

[GitHub] [solr] risdenk commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
risdenk commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011858823 ## solr/core/src/test/org/apache/solr/search/TestSmileRequest.java: ## @@ -71,7 +71,7 @@ public void testDistribJsonRequest() throws Exception { @Override

[GitHub] [solr] risdenk commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
risdenk commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011844656 ## solr/solrj/src/java/org/apache/solr/client/solrj/impl/SolrClientBuilder.java: ## @@ -46,6 +50,21 @@ public B withResponseParser(ResponseParser responseParser) {

[GitHub] [solr] mpetris commented on pull request #1156: [SOLR-16515] Remove synchronized access to cachedOrdMaps in SlowCompositeReaderWrapper

2022-11-02 Thread GitBox
mpetris commented on PR #1156: URL: https://github.com/apache/solr/pull/1156#issuecomment-1300514827 > Hey, this looks excellent! The code looks a bit deja-vu... do you think we could & should factorize out anything common? @dsmiley We could factorize out most of

[jira] [Commented] (SOLR-16367) Switch from CloudLegacySolrClient to CloudSolrClient

2022-11-02 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627725#comment-17627725 ] Bence Szabó commented on SOLR-16367: I see, having that in mind the code makes more sense now. Thank

[jira] [Commented] (SOLR-16367) Switch from CloudLegacySolrClient to CloudSolrClient

2022-11-02 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627696#comment-17627696 ] David Smiley commented on SOLR-16367: - Construction of the ClusterStateProvider is internal based on

[jira] [Commented] (SOLR-16367) Switch from CloudLegacySolrClient to CloudSolrClient

2022-11-02 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16367?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627674#comment-17627674 ] Bence Szabó commented on SOLR-16367: Maybe I am missing something but it seems to me there is no way

[jira] [Comment Edited] (SOLR-16512) Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627665#comment-17627665 ] Noble Paul edited comment on SOLR-16512 at 11/2/22 12:12 PM: - MapWriter is an

[jira] [Commented] (SOLR-16512) Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627665#comment-17627665 ] Noble Paul commented on SOLR-16512: --- MapWriter is an interface we use to serialize all objects in Solr.

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011650146 ## solr/solrj/src/test/org/apache/solr/client/solrj/embedded/SolrExampleStreamingBinaryTest.java: ## @@ -35,9 +34,15 @@ public class SolrExampleStreamingBinaryTest extends

[GitHub] [solr] epugh commented on a diff in pull request #1158: SOLR-16368: experiment with builder to simplify client creation

2022-11-02 Thread GitBox
epugh commented on code in PR #1158: URL: https://github.com/apache/solr/pull/1158#discussion_r1011647489 ## solr/solrj/src/test/org/apache/solr/client/solrj/embedded/SolrExampleStreamingBinaryTest.java: ## @@ -35,9 +34,15 @@ public class SolrExampleStreamingBinaryTest extends

[jira] [Commented] (SOLR-16512) Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17627655#comment-17627655 ] Eric Pugh commented on SOLR-16512: -- Out of curiosity, what does moving to MapWriter give us?   I read

[GitHub] [solr] noblepaul commented on pull request #1159: SOLR-16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
noblepaul commented on PR #1159: URL: https://github.com/apache/solr/pull/1159#issuecomment-1300178646 > Seems fine. Couldn't find review button, btw. I think if you go to the "files" tab , it's on the top right -- This is an automated message from the Apache Git Service. To

[jira] [Assigned] (SOLR-16512) Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16512?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Noble Paul reassigned SOLR-16512: - Assignee: Noble Paul > Eliminate noggit JSONWriter.Writable from Solr classes >

[GitHub] [solr] mkhludnev commented on pull request #1159: SOLR-16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
mkhludnev commented on PR #1159: URL: https://github.com/apache/solr/pull/1159#issuecomment-1299968926 Seems fine. Couldn't find review button, btw. -- 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

[jira] [Updated] (SOLR-16512) Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16512?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Noble Paul updated SOLR-16512: -- Summary: Eliminate noggit JSONWriter.Writable from Solr classes (was: Get rid of JSONWriter.Writable

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1299808362 For booleanQuery we have many tests in Lucene core and its own cache implementation. So it is three cases where it may break: - somebody calls getClauses() and modifies them (adding

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1299803390 Anyways, we should figure out if not the WrappedQuery is changing its hashCode, because this one was explicitely mutable. But the setter is never called from Solr (dead code according to

[GitHub] [solr] uschindler commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
uschindler commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1299799324 The clauses should not be modifyable from outside and I don't know any code doing this. BooleanQuery is designed to be immutable, the problem is more that there seems to be a getter that

[GitHub] [solr] ben-manes commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
ben-manes commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1299774688 Lucerne’s BooleanQuery clauseSets is externally mutable. The getClauses method returns a mutable collection (hashset) so the caller might add to it. I don’t see an occurrence, but some

[GitHub] [solr] DennisBerger1984 commented on pull request #1118: [SOLR-16489] CaffeineCache puts thread into infinite loop

2022-11-02 Thread GitBox
DennisBerger1984 commented on PR #1118: URL: https://github.com/apache/solr/pull/1118#issuecomment-1299722332 ![Screenshot_20221102_084057](https://user-images.githubusercontent.com/69144692/199430736-8210ff3c-0eb6-40d5-889d-aaad853d1bcb.png) It's a WrappedQuery Object. -- This is an

[GitHub] [solr] DennisBerger1984 commented on pull request #1156: [SOLR-16515] Remove synchronized access to cachedOrdMaps in SlowCompositeReaderWrapper

2022-11-02 Thread GitBox
DennisBerger1984 commented on PR #1156: URL: https://github.com/apache/solr/pull/1156#issuecomment-1299716722 ![Screenshot_20221102_084057](https://user-images.githubusercontent.com/69144692/199429777-e3f51cb5-406c-4a15-9441-db3537e389ac.png) It's a WrappedQuery object. -- This is

[GitHub] [solr] sonatype-lift[bot] commented on pull request #1159: SOLR: 16512 : Eliminate noggit JSONWriter.Writable from Solr classes

2022-11-02 Thread GitBox
sonatype-lift[bot] commented on PR #1159: URL: https://github.com/apache/solr/pull/1159#issuecomment-1299671194 :warning: **314 God Classes** were detected by Lift in this project. [Visit the Lift web