[jira] [Comment Edited] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646911#comment-17646911 ] Noble Paul edited comment on SOLR-16448 at 12/14/22 4:09 AM: -

[jira] [Comment Edited] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646911#comment-17646911 ] Noble Paul edited comment on SOLR-16448 at 12/14/22 4:08 AM: -

[jira] [Commented] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646911#comment-17646911 ] Noble Paul commented on SOLR-16448: --- {quote} he costs for building and running that clu

[jira] [Commented] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646910#comment-17646910 ] Shawn Heisey commented on SOLR-16448: - [~krisden] Maybe get rid of ALL the white spac

[GitHub] [solr] dsmiley commented on a diff in pull request #1239: SOLR-15787: Fix FileSystemConfigSetService test failure

2022-12-13 Thread GitBox
dsmiley commented on code in PR #1239: URL: https://github.com/apache/solr/pull/1239#discussion_r1047975716 ## solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java: ## @@ -238,9 +238,11 @@ public List getAllConfigFiles(String configName) throws IOException {

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1350071535 > Jetty thinks it stopped Are you sure that is not the new "start.jar" jetty that executes the STOP command? Cause we kind of fire up the jetty start jar to take jetty down... --

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1350065980 > I think now that we have a super fast way of reproducing, it should be easier to debug Unfortunately, this may be a dead end. What I thought was a reproduction may have been the effe

[jira] [Comment Edited] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646861#comment-17646861 ] Nazerke Seidan edited comment on SOLR-15787 at 12/13/22 11:14 PM: -

[jira] [Comment Edited] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646861#comment-17646861 ] Nazerke Seidan edited comment on SOLR-15787 at 12/13/22 11:13 PM: -

[jira] [Comment Edited] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646861#comment-17646861 ] Nazerke Seidan edited comment on SOLR-15787 at 12/13/22 11:11 PM: -

[jira] [Commented] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646861#comment-17646861 ] Nazerke Seidan commented on SOLR-15787: --- I created PR: [https://github.com/apache/s

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349954252 It is logical to think that it is related to Jetty 10. However, it it was pure Java change, we should see same behavior on local mac. That's why I felt the need to try from Docker to simula

[GitHub] [solr-operator] endzyme commented on pull request #473: Add a podDisruptionBudget for the whole cloud

2022-12-13 Thread GitBox
endzyme commented on PR #473: URL: https://github.com/apache/solr-operator/pull/473#issuecomment-1349943080 Thank you so much @HoustonPutman for this - Is there any way to avoid creating a PDB via the operator? -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349922398 Well after 6ba8d29 the logs are reporting something interesting... https://github.com/apache/solr/actions/runs/3689870566/jobs/6246292033#step:7:545 Jetty thinks it stopped

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349872685 Yea, I tried `java -jar server/start.jar --help` and get ``` --stop Sends a stop signal to the running Jetty instance. The running Jetty

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349806254 @janhoy at least from github actions the pid looks correct...: https://github.com/apache/solr/actions/runs/3689576579/jobs/6245669505 ``` # line 1858 ... # runner

[jira] [Commented] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Noble Paul (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646840#comment-17646840 ] Noble Paul commented on SOLR-16448: --- [~krisden] Our internal fork of Solr uses a gzip c

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349796157 > I think thats correct - as long as it isn't a `Z` or empty - it finds the stat column without the header - so it returned a result. Ok, might have been too quick to conclude there. An

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349793006 > If I debug even on my laptop, the command ps -o stat='' -p $SOLR_PID returns just a single S, not a PID I think thats correct - as long as it isn't a `Z` or empty - it finds the sta

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349784517 I think it is SOLR-16191 -- 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.

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349767958 > You see Solr itself running in PID 127. But the ps command targets PID 594, which is the PID of the stop script, which sleeps in a loop of 2 seconds. ok cool thanks @janhoy most like

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349728776 You see Solr itself running in PID 127. But the `ps` command targets PID 594, which is the PID of the stop script, which sleeps in a loop of 2 seconds. I think now that we have a super

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349710085 I exec'ed into the container during the 180 seconds it tried to stop and found this ``` docker exec -ti 00c425e5b1ca bash

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349707444 > I have NO idea why this passes locally before this change... It all works on my Macbook. But in Docker under Ubuntu it is definitely not happy -- This is an automated message from

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349704893 So I undid all my changes for debugging and so what is left is just the `stop.port` and `stop.key` parts. In theory this will just work. I have NO idea why this passes locally before t

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349704824 I reproduced this locally in Docker: ```bash docker run --rm -ti apache/solr-nightly:10.0.0-SNAPSHOT bash Unable to find image 'apache/solr-nightly:10.0.0-SNAPSHOT' locally 10.0

[GitHub] [solr] janhoy commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349683700 Yes, I was just looking into the same... -- 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 sp

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349677713 @janhoy https://github.com/apache/solr/pull/1223/commits/b6d814aace03837fcc61e2aa2f4315b93610487b may have fixed it! I wonder if its a Jetty 10.x change... I'm looking into https://g

[GitHub] [solr] magibney commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
magibney commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047678837 ## solr/CHANGES.txt: ## @@ -177,6 +177,12 @@ Other Changes * SOLR-16569: Add java system property to overseer queue size (Nick Ginther via noble) +=

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349607818 added some debug to see what https://github.com/apache/solr/pull/1223/commits/94d229c2e0ca3698590c9be51acfa143a190f362 is running. I would guess the failure is line 850 though since i

[jira] [Resolved] (SOLR-16586) Add logging at the beginning of searchHandler#handleRequestBody

2022-12-13 Thread Patson Luk (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16586?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patson Luk resolved SOLR-16586. --- Resolution: Won't Fix > Add logging at the beginning of searchHandler#handleRequestBody > ---

[jira] [Closed] (SOLR-16586) Add logging at the beginning of searchHandler#handleRequestBody

2022-12-13 Thread Patson Luk (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16586?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patson Luk closed SOLR-16586. - > Add logging at the beginning of searchHandler#handleRequestBody > -

[jira] [Commented] (SOLR-16586) Add logging at the beginning of searchHandler#handleRequestBody

2022-12-13 Thread Patson Luk (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646791#comment-17646791 ] Patson Luk commented on SOLR-16586: --- Since enabling debug on https://github.com/apache

[GitHub] [solr] patsonluk commented on pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
patsonluk commented on PR #1237: URL: https://github.com/apache/solr/pull/1237#issuecomment-1349536907 > Doesn't [this](https://github.com/apache/solr/blob/6869ec97c7a59cc6acd001eb592b98ea90679fb8/solr/core/src/java/org/apache/solr/core/SolrCore.java#L2861-L2864) allow to achieve essentiall

[GitHub] [solr] patsonluk closed pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
patsonluk closed pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody URL: https://github.com/apache/solr/pull/1237 -- 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 specif

[GitHub] [solr-operator] georgekaz opened a new issue, #505: security.json isn't exported to Zookeeper

2022-12-13 Thread GitBox
georgekaz opened a new issue, #505: URL: https://github.com/apache/solr-operator/issues/505 security.json isn't exported to Zookeeper because the logic `if [ ${#ZK_SECURITY_JSON} -lt 3 ]` [here](https://github.com/apache/solr-operator/blob/1a8661bab03656f55a3352835b8496d5e6757f9d/controller

[GitHub] [solr] dsmiley commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
dsmiley commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047582860 ## solr/CHANGES.txt: ## @@ -177,6 +177,12 @@ Other Changes * SOLR-16569: Add java system property to overseer queue size (Nick Ginther via noble) +==

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349496980 So @janhoy can you sanity check me - at the bottom of https://github.com/apache/solr/actions/runs/3658565463 are the logs from the integration test run. I don't see in `solr.log` or

[GitHub] [solr] dsmiley commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
dsmiley commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047579554 ## solr/CHANGES.txt: ## @@ -177,6 +177,12 @@ Other Changes * SOLR-16569: Add java system property to overseer queue size (Nick Ginther via noble) +==

[GitHub] [solr] risdenk commented on pull request #1223: SOLR-16584 BATS test fails stopping two instances

2022-12-13 Thread GitBox
risdenk commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1349463963 Well since `solr/bin/*` are the only files involved. That helps narrow it down considerably. I would guess its one of these changes... ``` # git log branch_9x..branch_9_1 -- solr/bi

[GitHub] [solr] patsonluk commented on a diff in pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
patsonluk commented on code in PR #1237: URL: https://github.com/apache/solr/pull/1237#discussion_r1047559663 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder( @Override public

[GitHub] [solr] magibney commented on pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
magibney commented on PR #1237: URL: https://github.com/apache/solr/pull/1237#issuecomment-1349417190 Doesn't [this](https://github.com/apache/solr/blob/6869ec97c7a59cc6acd001eb592b98ea90679fb8/solr/core/src/java/org/apache/solr/core/SolrCore.java#L2861-L2864) allow to achieve essentially t

[GitHub] [solr] patsonluk commented on a diff in pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
patsonluk commented on code in PR #1237: URL: https://github.com/apache/solr/pull/1237#discussion_r1047559663 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder( @Override public

[GitHub] [solr-operator] janhoy commented on issue #504: Probes for readiness and liveness should be different

2022-12-13 Thread GitBox
janhoy commented on issue #504: URL: https://github.com/apache/solr-operator/issues/504#issuecomment-1349385158 > Our common service (one endpoint for all nodes) has this set to true, while the headless service has this set to false. Thanks for the explanation. So there is a reason to

[GitHub] [solr] risdenk commented on a diff in pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
risdenk commented on code in PR #1237: URL: https://github.com/apache/solr/pull/1237#discussion_r1047549409 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder( @Override public v

[jira] [Commented] (SOLR-16448) State.json serialized with indent size 2, this takes lots of extra bytes for large collection

2022-12-13 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646763#comment-17646763 ] Kevin Risden commented on SOLR-16448: - So just some tangential thoughts related to th

[GitHub] [solr] patsonluk commented on a diff in pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
patsonluk commented on code in PR #1237: URL: https://github.com/apache/solr/pull/1237#discussion_r1047482742 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder( @Override public

[GitHub] [solr-operator] HoustonPutman commented on issue #504: Probes for readiness and liveness should be different

2022-12-13 Thread GitBox
HoustonPutman commented on issue #504: URL: https://github.com/apache/solr-operator/issues/504#issuecomment-1349006624 > So the liveness probe failing will cause k8s to kill the POD, and the readiness probe failing will cause traffic to be temporarily routed to other replicas, is that about

[GitHub] [solr] risdenk commented on a diff in pull request #1237: SOLR-16586: Log start of SearchHandler#handleRequestBody

2022-12-13 Thread GitBox
risdenk commented on code in PR #1237: URL: https://github.com/apache/solr/pull/1237#discussion_r1047382150 ## solr/core/src/java/org/apache/solr/handler/component/SearchHandler.java: ## @@ -326,6 +326,9 @@ protected ResponseBuilder newResponseBuilder( @Override public v

[GitHub] [solr-operator] janhoy commented on issue #504: Probes for readiness and liveness should be different

2022-12-13 Thread GitBox
janhoy commented on issue #504: URL: https://github.com/apache/solr-operator/issues/504#issuecomment-1348923782 So the liveness probe failing will cause k8s to kill the POD, and the readiness probe failing will cause traffic to be temporarily routed to other replicas, is that about right?

[GitHub] [solr-operator] janhoy merged pull request #499: Fix typo in readme

2022-12-13 Thread GitBox
janhoy merged PR #499: URL: https://github.com/apache/solr-operator/pull/499 -- 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.apa

[GitHub] [solr-operator] HoustonPutman commented on pull request #499: Fix typo in readme

2022-12-13 Thread GitBox
HoustonPutman commented on PR #499: URL: https://github.com/apache/solr-operator/pull/499#issuecomment-1348878380 Want to merge this @janhoy ? -- 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

[GitHub] [solr-operator] HoustonPutman merged pull request #473: Add a podDisruptionBudget for the whole cloud

2022-12-13 Thread GitBox
HoustonPutman merged PR #473: URL: https://github.com/apache/solr-operator/pull/473 -- 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...@s

[GitHub] [solr-operator] HoustonPutman closed issue #471: How to prevent node rotation behavior from causing cluster instability

2022-12-13 Thread GitBox
HoustonPutman closed issue #471: How to prevent node rotation behavior from causing cluster instability URL: https://github.com/apache/solr-operator/issues/471 -- 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

[GitHub] [solr-operator] HoustonPutman commented on issue #504: Probes for readiness and liveness should be different

2022-12-13 Thread GitBox
HoustonPutman commented on issue #504: URL: https://github.com/apache/solr-operator/issues/504#issuecomment-1348875539 We could use different commands. We can probably start mandating Solr 8 soon, and at that point we can use the solr healthcheck handler as the readiness check, and keep the

[GitHub] [solr] janhoy commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
janhoy commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047283207 ## solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java: ## @@ -2203,15 +2203,19 @@ public DocListAndSet getDocListAndSet(Query query, Sort lsort, int offset,

[GitHub] [solr] magibney commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
magibney commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047195884 ## solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java: ## @@ -2203,15 +2203,19 @@ public DocListAndSet getDocListAndSet(Query query, Sort lsort, int offset

[jira] [Comment Edited] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646631#comment-17646631 ] Nazerke Seidan edited comment on SOLR-15787 at 12/13/22 1:57 PM: --

[GitHub] [solr-operator] janhoy opened a new issue, #504: Probes for readiness and liveness should be different

2022-12-13 Thread GitBox
janhoy opened a new issue, #504: URL: https://github.com/apache/solr-operator/issues/504 We are using kyverno to enforce certain common k8s policies, and it complains about several things in Solr-operator. One is this one https://kyverno.io/policies/other/ensure_probes_different/ensu

[GitHub] [solr] magibney commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
magibney commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047173635 ## solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java: ## @@ -2203,15 +2203,19 @@ public DocListAndSet getDocListAndSet(Query query, Sort lsort, int offset

[jira] [Comment Edited] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646631#comment-17646631 ] Nazerke Seidan edited comment on SOLR-15787 at 12/13/22 1:22 PM: --

[jira] [Commented] (SOLR-15787) Implement FileSystemConfigSetService

2022-12-13 Thread Nazerke Seidan (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15787?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646631#comment-17646631 ] Nazerke Seidan commented on SOLR-15787: --- I am looking into it. I see the problem, i

[jira] [Resolved] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jan Høydahl resolved SOLR-16587. Resolution: Duplicate Closing as duplicate, as this is definitely same. Likely you will hit the sam

[GitHub] [solr] janhoy commented on a diff in pull request #1236: SOLR-16585: Fix NPE in MatchAllDocs pagination

2022-12-13 Thread GitBox
janhoy commented on code in PR #1236: URL: https://github.com/apache/solr/pull/1236#discussion_r1047094224 ## solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java: ## @@ -2203,15 +2203,19 @@ public DocListAndSet getDocListAndSet(Query query, Sort lsort, int offset,

[jira] [Commented] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646611#comment-17646611 ] Andrei Minin commented on SOLR-16587: - Not sure, you better know duplicate or not. My

[GitHub] [solr] EreMaijala commented on pull request #1238: Update caches-warming.adoc with a note about maxRamMB

2022-12-13 Thread GitBox
EreMaijala commented on PR #1238: URL: https://github.com/apache/solr/pull/1238#issuecomment-1348432560 For reference, I had documentCache with `maxRamMB="128"` consume 4.9 GB of memory. A quick web search indicated that others have stumbled on this issue before as well, so a nice little no

[jira] [Commented] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646602#comment-17646602 ] Jan Høydahl commented on SOLR-16587: I believe this may be a duplicate of SOLR-16585

[jira] [Comment Edited] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646596#comment-17646596 ] Andrei Minin edited comment on SOLR-16587 at 12/13/22 12:15 PM: ---

[jira] [Comment Edited] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646596#comment-17646596 ] Andrei Minin edited comment on SOLR-16587 at 12/13/22 12:14 PM: ---

[jira] [Commented] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646596#comment-17646596 ] Andrei Minin commented on SOLR-16587: - There is no need to create scripts - it is rep

[jira] [Updated] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrei Minin updated SOLR-16587: Attachment: Screenshot from 2022-12-13 15-10-11.png Screenshot from 2022-12-13 15-09

[GitHub] [solr] EreMaijala opened a new pull request, #1238: Update caches-warming.adoc with a note about maxRamMB

2022-12-13 Thread GitBox
EreMaijala opened a new pull request, #1238: URL: https://github.com/apache/solr/pull/1238 Add a note about documentCache not supporting maxRamMB setting properly. Since Lucene's RamUsageEstimator doesn't understand the Document objects, it will default to 256 bytes. Depending on the

[jira] [Commented] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Jira
[ https://issues.apache.org/jira/browse/SOLR-16587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17646592#comment-17646592 ] Jan Høydahl commented on SOLR-16587: Hi,   Can you also try to make a minimal repro

[jira] [Created] (SOLR-16587) Solr 9.1.0 last page retrieval error

2022-12-13 Thread Andrei Minin (Jira)
Andrei Minin created SOLR-16587: --- Summary: Solr 9.1.0 last page retrieval error Key: SOLR-16587 URL: https://issues.apache.org/jira/browse/SOLR-16587 Project: Solr Issue Type: Bug Securit

[GitHub] [solr] janhoy commented on pull request #1223: test bats

2022-12-13 Thread GitBox
janhoy commented on PR #1223: URL: https://github.com/apache/solr/pull/1223#issuecomment-1347898389 > I added `integrationTests` task to Check jenkins job for 9.1 and 9.x as well, so we can see if it fails for those as well, or if this is something introduced in main. Interesting - t