[jira] [Commented] (SOLR-15782) Configure custom node/container handlers in solr.xml

2023-11-02 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17782415#comment-17782415 ] David Smiley commented on SOLR-15782: - Perhaps we could establish that the following

Re: [PR] [9.x] SOLR-16943 Move Jetty HttpClient tracing into InstrumentedHttpListenerFactory [solr]

2023-11-02 Thread via GitHub
dsmiley commented on code in PR #2041: URL: https://github.com/apache/solr/pull/2041#discussion_r1381160995 ## solr/core/src/java/org/apache/solr/util/tracing/SolrJettyRequestCarrier.java: ## @@ -22,13 +22,13 @@ import java.util.Map; import org.eclipse.jetty.client.api.Request

Re: [PR] [9.x] SOLR-16943 Move Jetty HttpClient tracing into InstrumentedHttpListenerFactory [solr]

2023-11-02 Thread via GitHub
stillalex commented on PR #2041: URL: https://github.com/apache/solr/pull/2041#issuecomment-1791660915 did some manual testing and it looks ok -- 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

Re: [PR] [9.x] SOLR-16943 Move Jetty HttpClient tracing into InstrumentedHttpListenerFactory [solr]

2023-11-02 Thread via GitHub
stillalex commented on code in PR #2041: URL: https://github.com/apache/solr/pull/2041#discussion_r1380812192 ## solr/core/src/java/org/apache/solr/util/tracing/SolrHttpRequestCarrier.java: ## Review Comment: class renamed ## solr/core/src/java/org/apache/solr/ut

[jira] [Updated] (SOLR-16252) Server admins cannot override default value for UPDATE_TIMEOUT_SECS

2023-11-02 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alex Deparvu updated SOLR-16252: Issue Type: Improvement (was: Bug) > Server admins cannot override default value for UPDATE_TIMEOU

[jira] [Assigned] (SOLR-16252) Server admins cannot override default value for UPDATE_TIMEOUT_SECS

2023-11-02 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alex Deparvu reassigned SOLR-16252: --- Assignee: Alex Deparvu > Server admins cannot override default value for UPDATE_TIMEOUT_SECS

[PR] SOLR-16252 Server admins cannot override default value for UPDATE_TIMEOUT_SECS [solr]

2023-11-02 Thread via GitHub
stillalex opened a new pull request, #2061: URL: https://github.com/apache/solr/pull/2061 https://issues.apache.org/jira/browse/SOLR-16252 # Description Trivial change to introduce a system property `solr.schemaUpdateTimeoutSeconds` that defaults to original value `

[jira] [Commented] (SOLR-17040) UpdateLog refactor to use read/write locks

2023-11-02 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17782300#comment-17782300 ] Alex Deparvu commented on SOLR-17040: - thank you [~krisden] for the help here (especi

[jira] [Resolved] (SOLR-17040) UpdateLog refactor to use read/write locks

2023-11-02 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17040?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alex Deparvu resolved SOLR-17040. - Resolution: Won't Fix > UpdateLog refactor to use read/write locks >

Re: [PR] [WIP] SOLR-17040 UpdateLog refactor to use read/write locks [solr]

2023-11-02 Thread via GitHub
stillalex closed pull request #2026: [WIP] SOLR-17040 UpdateLog refactor to use read/write locks URL: https://github.com/apache/solr/pull/2026 -- 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 spe

Re: [PR] [WIP] SOLR-17040 UpdateLog refactor to use read/write locks [solr]

2023-11-02 Thread via GitHub
stillalex commented on PR #2026: URL: https://github.com/apache/solr/pull/2026#issuecomment-1791360174 closing. benchmarks do not show improvements on this change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[jira] [Commented] (SOLR-6213) StackOverflowException in Solr cloud's leader election

2023-11-02 Thread Alex Deparvu (Jira)
[ https://issues.apache.org/jira/browse/SOLR-6213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17782248#comment-17782248 ] Alex Deparvu commented on SOLR-6213: adding to a very very old jira, but I just saw th

Re: [PR] SOLR-17050: Save models as compacted json [solr]

2023-11-02 Thread via GitHub
epugh commented on PR #2030: URL: https://github.com/apache/solr/pull/2030#issuecomment-1791121376 we really don't want our users actually poking around in ZooKeeper...It really should be mediated by APIs and treated as an internal "implementation details"! -- This is an automate

Re: [PR] SOLR-16907: Reserve some 8.x security permission names [solr]

2023-11-02 Thread via GitHub
janhoy commented on code in PR #2040: URL: https://github.com/apache/solr/pull/2040#discussion_r1380254600 ## solr/core/src/java/org/apache/solr/security/Permission.java: ## @@ -52,18 +53,21 @@ static Permission load(Map m) { p.role = readValueAsSet(m, "role"); if (Per

Re: [PR] SOLR-16961 Remove deprecated CircuitBreakerManager in 10.0 [solr]

2023-11-02 Thread via GitHub
janhoy commented on PR #2059: URL: https://github.com/apache/solr/pull/2059#issuecomment-1790874704 Thanks for review, will merge tomorrow (friday) -- 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

Re: [PR] Capture build scans on ge.apache.org to benefit from deep build insights [solr]

2023-11-02 Thread via GitHub
dsmiley commented on PR #2042: URL: https://github.com/apache/solr/pull/2042#issuecomment-1790738569 Cool; I'll merge tomorrow if I don't get any +1 sooner. -- 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

Re: [PR] Capture build scans on ge.apache.org to benefit from deep build insights [solr]

2023-11-02 Thread via GitHub
clayburn commented on PR #2042: URL: https://github.com/apache/solr/pull/2042#issuecomment-1790701240 > It'd be nice if the target path for the scan could be configurable. As of now, the storage of the previous scan is really an implementation detail to support the `buildScanPublishPr

Re: [PR] Capture build scans on ge.apache.org to benefit from deep build insights [solr]

2023-11-02 Thread via GitHub
dsmiley commented on PR #2042: URL: https://github.com/apache/solr/pull/2042#issuecomment-1790693082 It'd be nice if the target path for the scan could be configurable. What's involved in having tests balance better across runners? Would that make sense in a separate PR? -- This i

[jira] [Updated] (SOLR-17064) Build: Enable org.gradle.caching by default

2023-11-02 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-17064?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Smiley updated SOLR-17064: Description: See SOLR-15603 for background but didn't enable it. Here we enable it. (was: See SOL

Re: [PR] SOLR-17050: Save models as compacted json [solr]

2023-11-02 Thread via GitHub
cpoerschke commented on PR #2030: URL: https://github.com/apache/solr/pull/2030#issuecomment-1790659304 > ... only the file is stored as compacted ... model-store endpoint will return a formatted JSON. ... You're right. I'd assumed users would 'look' directly in ZooKeeper and forgot

Re: [PR] Fix missing support in Consumer for other request types. [solr-sandbox]

2023-11-02 Thread via GitHub
markrmiller commented on PR #81: URL: https://github.com/apache/solr-sandbox/pull/81#issuecomment-1790415703 Been a little while since I've looked at that code, but ... Yes, the Kafka client is not thread safe. The design is a single thread polls the queue and hands off records to an

Re: [PR] Fix missing support in Consumer for other request types. [solr-sandbox]

2023-11-02 Thread via GitHub
sigram commented on PR #81: URL: https://github.com/apache/solr-sandbox/pull/81#issuecomment-1790406978 @markrmiller I'm not sure I understand everything of what's going on in `sendBatch` with workUnit. However, using executor here seems wrong because Kafka client is NOT thread-safe. If I'm