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

2024-10-14 Thread via GitHub
dsmiley commented on PR #2706: URL: https://github.com/apache/solr/pull/2706#issuecomment-2412356157 I think the next step is just some dependency version alignment? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] SOLR-17485 Remove CheckLoggingConfiguration; unnecessary [solr]

2024-10-14 Thread via GitHub
dsmiley opened a new pull request, #2763: URL: https://github.com/apache/solr/pull/2763 Also note we extend HttpFilter instead of ServletFilter. https://issues.apache.org/jira/browse/SOLR-17485 -- This is an automated message from the Apache Git Service. To respond to the message, p

[jira] [Updated] (SOLR-17485) Remove Slf4j ClassNotFoundException avoidance checks

2024-10-14 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17485?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated SOLR-17485: -- Labels: pull-request-available (was: ) > Remove Slf4j ClassNotFoundException avoidance checks

[jira] [Commented] (SOLR-17414) multiThreaded=true can result in RejectedExecutionException

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889319#comment-17889319 ] ASF subversion and git services commented on SOLR-17414: Commit 8

[jira] [Commented] (SOLR-13350) Explore collector managers for multi-threaded search

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-13350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889321#comment-17889321 ] ASF subversion and git services commented on SOLR-13350: Commit 8

[jira] [Commented] (SOLR-17298) Multithreaded search breaks limits, and possibly other things

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889320#comment-17889320 ] ASF subversion and git services commented on SOLR-17298: Commit 8

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

2024-10-14 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated SOLR-17456: -- Labels: pull-request-available (was: ) > TransactionLog NPE > -- > >

[PR] SOLR-17456: TransactionLog ctor integrity [solr]

2024-10-14 Thread via GitHub
dsmiley opened a new pull request, #2762: URL: https://github.com/apache/solr/pull/2762 The TransactionLog constructor can't handle an existing file being present; it shouldn't be there. Should throw an exception in this case, NOT log a warning, leaving the object in a partially constructe

Re: [PR] SOLR-17456: TransactionLog ctor integrity [solr]

2024-10-14 Thread via GitHub
dsmiley commented on code in PR #2762: URL: https://github.com/apache/solr/pull/2762#discussion_r1800172613 ## solr/core/src/java/org/apache/solr/update/TransactionLog.java: ## @@ -238,9 +240,6 @@ protected TransactionLog( } success = true; - - assert Object

Re: [PR] Bump up Java version to 21 [solr]

2024-10-14 Thread via GitHub
iamsanjay commented on PR #2682: URL: https://github.com/apache/solr/pull/2682#issuecomment-2412897758 If I set sourceCompatibility at the same level as "--release" flag then It worked, which is basically set it the package level (main or test) instead of Project. Now after this, We can use

Re: [PR] Bump up Java version to 21 [solr]

2024-10-14 Thread via GitHub
iamsanjay commented on PR #2682: URL: https://github.com/apache/solr/pull/2682#issuecomment-2412900492 Documentation from Intellij is very much relevant at package level as well. For language level it looks at sourceCompatibility. > Language level: the language level settings are app

Re: [PR] Bump up Java version to 21 [solr]

2024-10-14 Thread via GitHub
dsmiley commented on PR #2682: URL: https://github.com/apache/solr/pull/2682#issuecomment-2412901338 I see; it works! -- 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 uns

Re: [PR] Bump up Java version to 21 [solr]

2024-10-14 Thread via GitHub
dsmiley commented on PR #2682: URL: https://github.com/apache/solr/pull/2682#issuecomment-2412845114 As I mentioned, IntelliJ isn't recognizing the language distinction between tests vs non-tests so I recommend simply using the same language per module. Other than that, I suppose it's read

Re: [PR] Bump up Java version to 21 [solr]

2024-10-14 Thread via GitHub
iamsanjay commented on PR #2682: URL: https://github.com/apache/solr/pull/2682#issuecomment-2412913512 We may need to remove some of the compiler options as well. As they may not be required anymore with the upgrade, also may be two different sets of compiler options -- one for JDK 17 and a

Re: [PR] SOLR-17494: Remove language specific writer types [solr]

2024-10-14 Thread via GitHub
janhoy commented on PR #2758: URL: https://github.com/apache/solr/pull/2758#issuecomment-2412422930 Deprecate as notice in ref guide. Also we have a separate deprecation logging mechanism. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Update org.slf4j:* to v2.0.16 [solr]

2024-10-14 Thread via GitHub
github-actions[bot] commented on PR #2637: URL: https://github.com/apache/solr/pull/2637#issuecomment-2412549240 This PR has had no activity for 60 days and is now labeled as stale. Any new activity will remove the stale label. To attract more reviewers, please tag people who might be fam

[jira] [Commented] (SOLR-17298) Multithreaded search breaks limits, and possibly other things

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889249#comment-17889249 ] ASF subversion and git services commented on SOLR-17298: Commit 7

[jira] [Commented] (SOLR-17414) multiThreaded=true can result in RejectedExecutionException

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889248#comment-17889248 ] ASF subversion and git services commented on SOLR-17414: Commit 7

[jira] [Commented] (SOLR-13350) Explore collector managers for multi-threaded search

2024-10-14 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-13350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889250#comment-17889250 ] ASF subversion and git services commented on SOLR-13350: Commit 7

Re: [PR] SOLR-17414: Multi-threaded search: don't throw RejectedExecutionException [solr]

2024-10-14 Thread via GitHub
dsmiley merged PR #2701: URL: https://github.com/apache/solr/pull/2701 -- 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-17495: Change CLI delete command to not delete configs by default. [solr]

2024-10-14 Thread via GitHub
epugh commented on PR #2761: URL: https://github.com/apache/solr/pull/2761#issuecomment-2411861328 Leaving "tutorial-films.adoc" alone, as it uses `_default` and we wouldn't want to delete it. -- This is an automated message from the Apache Git Service. To respond to the message, please l

[jira] [Commented] (SOLR-17413) UpdateLog Replay can throw ConcurrentModificationException from sharing the request

2024-10-14 Thread Jason Gerlowski (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17889281#comment-17889281 ] Jason Gerlowski commented on SOLR-17413: bq. This should be somewhat straightforw

Re: [PR] Add link to blog post on C/C NA BOF [solr-site]

2024-10-14 Thread via GitHub
epugh commented on code in PR #129: URL: https://github.com/apache/solr-site/pull/129#discussion_r1799876019 ## content/pages/blogposts/2024-10-14-community-over-code-na-birds-of-a-feather.md: ## @@ -0,0 +1,11 @@ +Title: Community Over Code NA 2024 Apache Lucene/Solr Birds Of A

Re: [PR] Add link to blog post on C/C NA BOF [solr-site]

2024-10-14 Thread via GitHub
epugh commented on PR #129: URL: https://github.com/apache/solr-site/pull/129#issuecomment-2411871664 thanks for feedback..I'd love a +1 or even a merge to get this done and out the door ;-) -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] SOLR-17453: Leverage waitForState() instead of busy waiting [solr]

2024-10-14 Thread via GitHub
epugh commented on code in PR #2737: URL: https://github.com/apache/solr/pull/2737#discussion_r1799884914 ## solr/core/src/test/org/apache/solr/cloud/TestRebalanceLeaders.java: ## @@ -604,74 +572,61 @@ private void forceUpdateCollectionStatus() { // Since we have to restart

Re: [PR] Add link to blog post on C/C NA BOF [solr-site]

2024-10-14 Thread via GitHub
gerlowskija commented on code in PR #129: URL: https://github.com/apache/solr-site/pull/129#discussion_r1799405713 ## content/pages/blogposts/2024-10-14-community-over-code-na-birds-of-a-feather.md: ## @@ -0,0 +1,11 @@ +Title: Community Over Code NA 2024 Apache Lucene/Solr Birds

Re: [PR] Add link to blog post on C/C NA BOF [solr-site]

2024-10-14 Thread via GitHub
gerlowskija commented on PR #129: URL: https://github.com/apache/solr-site/pull/129#issuecomment-2411131166 > I saw the previous BoF blogpost (https://github.com/apache/solr-site/pull/110) had a build failure as well. Is it the same as here and not really an issue? Looks like the "ac

[jira] [Updated] (SOLR-17150) Create MemQueryLimit implementation

2024-10-14 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrzej Bialecki updated SOLR-17150: Fix Version/s: 9.8 > Create MemQueryLimit implementation >

[jira] [Resolved] (SOLR-17150) Create MemQueryLimit implementation

2024-10-14 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrzej Bialecki resolved SOLR-17150. - Resolution: Fixed > Create MemQueryLimit implementation > ---

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

2024-10-14 Thread via GitHub
malliaridis commented on code in PR #2712: URL: https://github.com/apache/solr/pull/2712#discussion_r1799446120 ## solr/packaging/test/test_status.bats: ## @@ -37,18 +37,27 @@ teardown() { solr stop run solr status assert_output --partial "No Solr nodes are running." +}

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

2024-10-14 Thread via GitHub
malliaridis commented on code in PR #2712: URL: https://github.com/apache/solr/pull/2712#discussion_r1799446120 ## solr/packaging/test/test_status.bats: ## @@ -37,18 +37,27 @@ teardown() { solr stop run solr status assert_output --partial "No Solr nodes are running." +}

Re: [PR] SOLR-17495: Change CLI delete command to not delete configs by default. [solr]

2024-10-14 Thread via GitHub
epugh commented on code in PR #2761: URL: https://github.com/apache/solr/pull/2761#discussion_r1799860269 ## solr/core/src/java/org/apache/solr/cli/DeleteTool.java: ## @@ -198,13 +187,13 @@ protected void deleteCollection(CloudSolrClient cloudSolrClient, CommandLine cli

Re: [PR] SOLR-17453: Leverage waitForState() instead of busy waiting [solr]

2024-10-14 Thread via GitHub
dsmiley commented on PR #2737: URL: https://github.com/apache/solr/pull/2737#issuecomment-2411992713 A line of javadocs on TimeOut recommending waitForState would be good but perhaps arguably a distraction to a general utility -- :shrug: whatever. I don't think we'll backslide much if all/

Re: [PR] SOLR-17495: Change CLI delete command to not delete configs by default. [solr]

2024-10-14 Thread via GitHub
malliaridis commented on code in PR #2761: URL: https://github.com/apache/solr/pull/2761#discussion_r1799701043 ## solr/core/src/java/org/apache/solr/cli/DeleteTool.java: ## @@ -198,13 +187,13 @@ protected void deleteCollection(CloudSolrClient cloudSolrClient, CommandLine cli