[GitHub] [solr] dsmiley commented on a diff in pull request #975: Allow JAX-RS v2 API definition

2022-08-22 Thread GitBox
dsmiley commented on code in PR #975: URL: https://github.com/apache/solr/pull/975#discussion_r952142394 ## solr/core/src/java/org/apache/solr/api/V2HttpCall.java: ## @@ -157,20 +168,16 @@ public void call(SolrQueryRequest req, SolrQueryResponse rsp) { origCorename = p

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

2022-08-22 Thread Shubanker (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shubanker updated SOLR-16350: - Description: While trying to setup SOLR 9 I noticed the response takes longer to complete if the interne

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

2022-08-22 Thread Shubanker (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shubanker updated SOLR-16350: - Attachment: image-2022-08-23-10-03-45-870.png > http response is delayed when solr is running in slow/res

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

2022-08-22 Thread Shubanker (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shubanker updated SOLR-16350: - Attachment: image-2022-08-23-10-01-40-456.png > http response is delayed when solr is running in slow/res

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

2022-08-22 Thread Shubanker (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shubanker updated SOLR-16350: - Attachment: image-2022-08-23-09-59-25-775.png > http response is delayed when solr is running in slow/res

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

2022-08-22 Thread Shubanker (Jira)
Shubanker created SOLR-16350: Summary: http response is delayed when solr is running in slow/restricted network environment Key: SOLR-16350 URL: https://issues.apache.org/jira/browse/SOLR-16350 Project: S

[GitHub] [solr] dsmiley commented on pull request #980: always(?) use SolrCore.getLatestSchema in RealTimeGetComponent

2022-08-22 Thread GitBox
dsmiley commented on PR #980: URL: https://github.com/apache/solr/pull/980#issuecomment-1223490615 For code activated from a request, I think the code should use a consistent schema for that request. Thus I think RTG should be using req.getSchema() in those 10 places. -- This is an auto

[GitHub] [solr] epugh commented on pull request #974: Mention REINDEXCOLLECTION in Reindexing docs

2022-08-22 Thread GitBox
epugh commented on PR #974: URL: https://github.com/apache/solr/pull/974#issuecomment-1222804823 Nice fix! -- 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

[GitHub] [solr] epugh merged pull request #974: Mention REINDEXCOLLECTION in Reindexing docs

2022-08-22 Thread GitBox
epugh merged PR #974: URL: https://github.com/apache/solr/pull/974 -- 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.org F

[GitHub] [solr] dsmiley commented on pull request #943: SOLR-15342: Separate out a SolrJ-Zookeeper module

2022-08-22 Thread GitBox
dsmiley commented on PR #943: URL: https://github.com/apache/solr/pull/943#issuecomment-1222663050 I think it's done and I've addressed comments. I would love to see this merged soon so as to unblock other efforts that touch lots of files. If nobody has further review comments, I'll merge

[jira] [Resolved] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shawn Heisey resolved SOLR-16327. - Resolution: Not A Bug Resolving as not a bug, since evidence says Solr is behaving correctly. >

[jira] [Commented] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17583092#comment-17583092 ] Shawn Heisey commented on SOLR-16327: - I agree.  It looks to me like a bug in haproxy

[GitHub] [solr] cpoerschke opened a new pull request, #980: always(?) use SolrCore.getLatestSchema in RealTimeGetComponent

2022-08-22 Thread GitBox
cpoerschke opened a new pull request, #980: URL: https://github.com/apache/solr/pull/980 The `RealTimeGetComponent` uses `getLatestSchema` in 12 places and `getSchema` in 2 places. Intuitively I would expect `getSchema` to be used throughout since it as per https://github.com/apache/

[GitHub] [solr] heythm commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
heythm commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951327744 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * * @re

[jira] [Commented] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17583053#comment-17583053 ] Jan Høydahl commented on SOLR-16327: The Developer Tools reports what the browser is

[GitHub] [solr] uschindler commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
uschindler commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951543778 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * *

[GitHub] [solr] madrob opened a new pull request, #979: SOLR-16349 check for properties file before attempt to open

2022-08-22 Thread GitBox
madrob opened a new pull request, #979: URL: https://github.com/apache/solr/pull/979 https://issues.apache.org/jira/browse/SOLR-16349 I'm not sure if a real production scenario would prefer a logged exception here, but the behavior is unchanged. -- This is an automated message from

[GitHub] [solr] uschindler commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
uschindler commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951532817 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * *

[jira] [Created] (SOLR-16349) TestHarness complains about missing core.properties file

2022-08-22 Thread Mike Drob (Jira)
Mike Drob created SOLR-16349: Summary: TestHarness complains about missing core.properties file Key: SOLR-16349 URL: https://issues.apache.org/jira/browse/SOLR-16349 Project: Solr Issue Type: Bug

[GitHub] [solr] uschindler commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
uschindler commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951532817 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * *

[GitHub] [solr] uschindler commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
uschindler commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951526611 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * *

[GitHub] [solr] risdenk commented on pull request #840: SOLR-16187: ExecutorUtil#awaitTermination shouldn't wait forever

2022-08-22 Thread GitBox
risdenk commented on PR #840: URL: https://github.com/apache/solr/pull/840#issuecomment-1222455223 @madrob / @magibney I updated this PR to latest main and added throwing an exception if the pool couldn't be shutdown. -- This is an automated message from the Apache Git Service. To respond

[jira] [Commented] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17582990#comment-17582990 ] Shawn Heisey commented on SOLR-16327: - The haproxy log shows a 503 error for both req

[jira] [Updated] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shawn Heisey updated SOLR-16327: Attachment: image-2022-08-22-07-47-00-751.png > Admin UI reports permission error if the request it

[jira] [Updated] (SOLR-16327) Admin UI reports permission error if the request it made results in ANY exception

2022-08-22 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shawn Heisey updated SOLR-16327: Attachment: image-2022-08-22-07-47-00-809.png > Admin UI reports permission error if the request it

[GitHub] [solr] dsmiley commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
dsmiley commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951401025 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * * @r

[GitHub] [solr] heythm commented on a diff in pull request #962: SOLR-16296: Load elevate.xml in a more secure way

2022-08-22 Thread GitBox
heythm commented on code in PR #962: URL: https://github.com/apache/solr/pull/962#discussion_r951327744 ## solr/core/src/java/org/apache/solr/handler/component/QueryElevationComponent.java: ## @@ -376,10 +377,12 @@ protected long getConfigVersion(SolrCore core) { * * @re

[jira] [Commented] (SOLR-16344) PlacementPlugin gets a request without a collection

2022-08-22 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17582910#comment-17582910 ] Noble Paul commented on SOLR-16344: --- lemme take a look > PlacementPlugin gets a reques

[GitHub] [solr-operator] alexangas opened a new issue, #467: Should the "OOM killer script" be added as default

2022-08-22 Thread GitBox
alexangas opened a new issue, #467: URL: https://github.com/apache/solr-operator/issues/467 Reviewing [Choosing Memory Heap Settings](https://solr.apache.org/guide/8_11/jvm-settings.html#choosing-memory-heap-settings) and wondering if it makes sense to set the "OOM killer script" as a defau