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

2024-10-12 Thread via GitHub
epugh commented on PR #2758: URL: https://github.com/apache/solr/pull/2758#issuecomment-2408820754 Thoughts on how to mark it "deprecated" in 9x? In some places it makes sense to put out a log message. What would being deprecated mean here? Just a PR that adds `@deprecated` in the `br

[jira] [Created] (SOLR-17496) Reword messaging about Zookeeper in startup logs

2024-10-12 Thread Eric Pugh (Jira)
Eric Pugh created SOLR-17496: Summary: Reword messaging about Zookeeper in startup logs Key: SOLR-17496 URL: https://issues.apache.org/jira/browse/SOLR-17496 Project: Solr Issue Type: Sub-task

[jira] [Updated] (SOLR-17496) Reword messaging about Zookeeper in startup logs

2024-10-12 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17496?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Pugh updated SOLR-17496: - Description: We output text like:   TARTING EMBEDDED ENSEMBLE ZOOKEEPER SERVER at port 9983, listening

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

2024-10-12 Thread via GitHub
gerlowskija commented on code in PR #2758: URL: https://github.com/apache/solr/pull/2758#discussion_r1797929696 ## solr/core/src/test/org/apache/solr/response/JSONWriterTest.java: ## @@ -53,24 +53,17 @@ private void jsonEq(String expected, String received) { } @Test - p

Re: [PR] SOLR-17489: CLI: Deprecate variations on solr urls options [solr]

2024-10-12 Thread via GitHub
malliaridis commented on code in PR #2756: URL: https://github.com/apache/solr/pull/2756#discussion_r1797711145 ## solr/core/src/java/org/apache/solr/cli/PostLogsTool.java: ## @@ -66,24 +67,46 @@ public List getOptions() { return List.of( Option.builder("url")

Re: [PR] SOLR-17488: CLI: Resolve -d conflicts [solr]

2024-10-12 Thread via GitHub
epugh commented on code in PR #2754: URL: https://github.com/apache/solr/pull/2754#discussion_r1797712401 ## solr/bin/solr.cmd: ## @@ -1520,7 +1521,9 @@ for %%a in (%*) do ( ) else ( set "option!option!=%%a" if "!option!" equ "-d" set "SOLR_SERVER_DIR=%%a" +

Re: [PR] SOLR-17489: CLI: Deprecate variations on solr urls options [solr]

2024-10-12 Thread via GitHub
epugh commented on code in PR #2756: URL: https://github.com/apache/solr/pull/2756#discussion_r1797734402 ## solr/core/src/test/org/apache/solr/cli/PostToolTest.java: ## @@ -85,8 +85,10 @@ public void testBasicRun() throws Exception { String[] args = { "post", -

[jira] [Commented] (SOLR-17488) CLI: Resolve -d conflicts

2024-10-12 Thread Christos Malliaridis (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17888913#comment-17888913 ] Christos Malliaridis commented on SOLR-17488: - Right now, the user has to pro

Re: [PR] SOLR-17488: Parse long format for auth command in bin/solr.cmd [solr]

2024-10-12 Thread via GitHub
malliaridis commented on code in PR #2760: URL: https://github.com/apache/solr/pull/2760#discussion_r1797721419 ## solr/bin/solr.cmd: ## @@ -1519,11 +1519,11 @@ for %%a in (%*) do ( if "!arg:~0,1!" equ "-" set "option=!arg!" ) else ( set "option!option!=%%a" -

[jira] [Commented] (SOLR-17478) ZkTestServer bugs can cause 30sec test pauses

2024-10-12 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17888954#comment-17888954 ] David Smiley commented on SOLR-17478: - +1 simply trusting this works > ZkTestServer

Re: [PR] SOLR-17488: CLI: Resolve -d conflicts [solr]

2024-10-12 Thread via GitHub
malliaridis commented on code in PR #2754: URL: https://github.com/apache/solr/pull/2754#discussion_r1797701186 ## solr/bin/solr.cmd: ## @@ -1520,7 +1521,9 @@ for %%a in (%*) do ( ) else ( set "option!option!=%%a" if "!option!" equ "-d" set "SOLR_SERVER_DIR=%%a

[jira] [Commented] (SOLR-17488) CLI: Resolve -d conflicts

2024-10-12 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17888923#comment-17888923 ] Eric Pugh commented on SOLR-17488: -- I think this deserves it's own JIRA...  I like the i

[jira] [Commented] (SOLR-17488) CLI: Resolve -d conflicts

2024-10-12 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17888924#comment-17888924 ] Eric Pugh commented on SOLR-17488: -- See SOLR-17495 > CLI: Resolve -d conflicts > --

[jira] [Created] (SOLR-17495) Change CLI DeleteTool deleting of configs to opt in instead of default behavior

2024-10-12 Thread Eric Pugh (Jira)
Eric Pugh created SOLR-17495: Summary: Change CLI DeleteTool deleting of configs to opt in instead of default behavior Key: SOLR-17495 URL: https://issues.apache.org/jira/browse/SOLR-17495 Project: Solr

[jira] [Commented] (SOLR-17488) CLI: Resolve -d conflicts

2024-10-12 Thread Christos Malliaridis (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17888928#comment-17888928 ] Christos Malliaridis commented on SOLR-17488: - Sounds good. Thanks for creati

[PR] SOLR-17488: Parse long format for auth command in bin/solr.cmd [solr]

2024-10-12 Thread via GitHub
epugh opened a new pull request, #2760: URL: https://github.com/apache/solr/pull/2760 https://issues.apache.org/jira/browse/SOLR-17488 Bug fix based on feedback from @malliaridis. We need to take a pass through solr.cmd and see if we can reduce the custom per option parsing!

Re: [PR] SOLR-17488: CLI: Resolve -d conflicts [solr]

2024-10-12 Thread via GitHub
epugh commented on code in PR #2754: URL: https://github.com/apache/solr/pull/2754#discussion_r1797715501 ## solr/bin/solr.cmd: ## @@ -1520,7 +1521,9 @@ for %%a in (%*) do ( ) else ( set "option!option!=%%a" if "!option!" equ "-d" set "SOLR_SERVER_DIR=%%a" +

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

2024-10-12 Thread via GitHub
epugh opened a new pull request, #2761: URL: https://github.com/apache/solr/pull/2761 https://issues.apache.org/jira/browse/SOLR-17495 # Description See jira # Solution Makes --delete-config a opt in flag that specifies the user wants to delete their config al

[jira] [Updated] (SOLR-17495) Change CLI DeleteTool deleting of configs to opt in instead of default behavior

2024-10-12 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17495?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated SOLR-17495: -- Labels: pull-request-available (was: ) > Change CLI DeleteTool deleting of configs to opt in i

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

2024-10-12 Thread via GitHub
epugh commented on PR #2761: URL: https://github.com/apache/solr/pull/2761#issuecomment-2408625057 @malliaridis -- 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 unsubscr

Re: [PR] Update dependency com.lmax:disruptor to v4 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #1990: URL: https://github.com/apache/solr/pull/1990#issuecomment-2408749803 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

Re: [PR] SOLR-17405: allow a single thread to reestablish ZK session [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2644: URL: https://github.com/apache/solr/pull/2644#issuecomment-2408749687 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

Re: [PR] Update dependency org.hsqldb:hsqldb to v2.7.3 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2608: URL: https://github.com/apache/solr/pull/2608#issuecomment-2408749712 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

Re: [PR] Update dependency com.github.spotbugs:spotbugs-annotations to v4.8.6 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2129: URL: https://github.com/apache/solr/pull/2129#issuecomment-2408749793 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

Re: [PR] Update dependency org.carrot2:carrot2-core to v4.6.0 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2130: URL: https://github.com/apache/solr/pull/2130#issuecomment-2408749780 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

Re: [PR] Update dependency jakarta.ws.rs:jakarta.ws.rs-api to v4 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2441: URL: https://github.com/apache/solr/pull/2441#issuecomment-2408749753 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

Re: [PR] Update org.hamcrest:* to v3 (major) [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2617: URL: https://github.com/apache/solr/pull/2617#issuecomment-2408749700 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

Re: [PR] Update org.apache.tika:* to v2 (major) [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2583: URL: https://github.com/apache/solr/pull/2583#issuecomment-2408749725 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

Re: [PR] Update dependency biz.aQute.bnd:biz.aQute.bnd.annotation to v7 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2581: URL: https://github.com/apache/solr/pull/2581#issuecomment-2408749738 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

Re: [PR] Update org.glassfish.hk2*:* to v3.1.1 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2268: URL: https://github.com/apache/solr/pull/2268#issuecomment-2408749762 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

Re: [PR] Update io.grpc:grpc-* to v1.66.0 [solr]

2024-10-12 Thread via GitHub
github-actions[bot] commented on PR #2639: URL: https://github.com/apache/solr/pull/2639#issuecomment-2408749695 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