[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #753: SOLR-16005 added new feature in to schema screen: add new field type

2022-07-05 Thread GitBox
sonatype-lift[bot] commented on code in PR #753: URL: https://github.com/apache/solr/pull/753#discussion_r914326117 ## solr/webapp/web/js/angular/controllers/schema.js: ## @@ -289,6 +302,42 @@ solrAdminApp.controller('SchemaController', } });

[GitHub] [solr] ackepenek commented on pull request #753: SOLR-16005 added new feature in to schema screen: add new field type

2022-07-05 Thread GitBox
ackepenek commented on PR #753: URL: https://github.com/apache/solr/pull/753#issuecomment-1175665038 @epugh I fixed the PR as you mentioned. You can see new screenshots below: ![Screenshot from 2022-07-06 04-16-48](https://user-images.githubusercontent.com/1234305/177444798-d89141

[GitHub] [solr-operator] HoustonPutman opened a new pull request, #454: Use the Solr image as the default Prom Exp image

2022-07-05 Thread GitBox
HoustonPutman opened a new pull request, #454: URL: https://github.com/apache/solr-operator/pull/454 Resolves #386 Still todo: - [ ] Documentation & Upgrade Notes - [ ] Tests (Unit & Integration) -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [solr-operator] HoustonPutman commented on pull request #453: Add app-protocol for all services

2022-07-05 Thread GitBox
HoustonPutman commented on PR #453: URL: https://github.com/apache/solr-operator/pull/453#issuecomment-1175419928 @johgoe if you have time, please take a look at this PR! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [solr-operator] HoustonPutman commented on issue #427: Problems with istio and prometheus

2022-07-05 Thread GitBox
HoustonPutman commented on issue #427: URL: https://github.com/apache/solr-operator/issues/427#issuecomment-1175419265 Note, it looks like there is a difference between the `http` and `http2` **appProtocol**. We will default to `http` now for unsecured services, but we might want to look in

[GitHub] [solr-operator] HoustonPutman opened a new pull request, #453: Add app-protocol for all services

2022-07-05 Thread GitBox
HoustonPutman opened a new pull request, #453: URL: https://github.com/apache/solr-operator/pull/453 Resolves #427 -- 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 unsubs

[jira] [Resolved] (SOLR-16276) Review redundant local variables

2022-07-05 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Pugh resolved SOLR-16276. -- Fix Version/s: 9.1 main (10.0) Resolution: Fixed > Review redundant local var

[GitHub] [solr] epugh merged pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh merged PR #924: URL: https://github.com/apache/solr/pull/924 -- 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

[jira] [Commented] (SOLR-16276) Review redundant local variables

2022-07-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562794#comment-17562794 ] ASF subversion and git services commented on SOLR-16276: Commit e

[jira] [Commented] (SOLR-16276) Review redundant local variables

2022-07-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562789#comment-17562789 ] ASF subversion and git services commented on SOLR-16276: Commit e

[GitHub] [solr-operator] HoustonPutman opened a new pull request, #452: Make new TLSTermination option, with or w/o secret

2022-07-05 Thread GitBox
HoustonPutman opened a new pull request, #452: URL: https://github.com/apache/solr-operator/pull/452 Resolves #437 - Use old option to default new `Spec.solrAddressability.external.ingressTLSTermination.tlsSecret`. - Deprecate old `Spec.solrAddressability.external.ingressTLSTermin

[GitHub] [solr] cpoerschke commented on a diff in pull request #929: SOLR-16280: simplify for loop

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #929: URL: https://github.com/apache/solr/pull/929#discussion_r914001900 ## solr/core/src/test/org/apache/solr/schema/TestPointFields.java: ## @@ -5289,9 +5289,9 @@ private void doTestSetQueries(String fieldName, String[] values, boolean multi

[GitHub] [solr] risdenk commented on pull request #585: SOLR-15955 Update Jetty dependency to 10

2022-07-05 Thread GitBox
risdenk commented on PR #585: URL: https://github.com/apache/solr/pull/585#issuecomment-1175293623 Is there more to do here? I know @anshumg asked that Jetty 9.4.x upgrade be superseded by this change. There have been a few more Jetty 9.4.x and 10.x releases. -- This is an automated mess

[jira] [Commented] (SOLR-16221) Solr Jenkins CI - s390x support

2022-07-05 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562759#comment-17562759 ] Uwe Schindler commented on SOLR-16221: -- It looks like the Lucene Job passed. So cong

[GitHub] [solr] epugh commented on pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on PR #924: URL: https://github.com/apache/solr/pull/924#issuecomment-1175236948 running all the tests again, and tjhen will merge. Thanks @cpoerschke for the efforts! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Commented] (SOLR-16221) Solr Jenkins CI - s390x support

2022-07-05 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562727#comment-17562727 ] Uwe Schindler commented on SOLR-16221: -- Ok, one job is starting > Solr Jenkins

[jira] [Commented] (SOLR-16221) Solr Jenkins CI - s390x support

2022-07-05 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562726#comment-17562726 ] Uwe Schindler commented on SOLR-16221: -- Now all nodes are down, the label "lucene||s

[GitHub] [solr] cpoerschke commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913939802 ## solr/core/src/test/org/apache/solr/schema/TestCloudSchemaless.java: ## @@ -63,7 +63,8 @@ protected String getCloudSolrConfig() { @Override public SortedMap getE

[GitHub] [solr] cpoerschke commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913914168 ## solr/core/src/test/org/apache/solr/update/PeerSyncWithIndexFingerprintCachingTest.java: ## @@ -110,6 +110,6 @@ void assertSync(SolrClient client, int numVersions, boole

[GitHub] [solr] cpoerschke commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913913233 ## solr/core/src/test/org/apache/solr/update/PeerSyncWithBufferUpdatesTest.java: ## @@ -227,6 +227,6 @@ void assertSync(SolrClient client, int numVersions, boolean expect

[GitHub] [solr] cpoerschke commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913912286 ## solr/core/src/test/org/apache/solr/update/MaxSizeAutoCommitTest.java: ## @@ -204,46 +201,44 @@ public void testDeletes() throws Exception { updateHandler,

[GitHub] [solr] epugh commented on a diff in pull request #932: SOLR-15182: Remove remaining apispec files

2022-07-05 Thread GitBox
epugh commented on code in PR #932: URL: https://github.com/apache/solr/pull/932#discussion_r913892056 ## solr/core/src/test/org/apache/solr/handler/admin/V2ApiMappingTest.java: ## @@ -204,11 +204,12 @@ public String getHttpMethod() { return queryRequestCaptor.getValue().ge

[GitHub] [solr] epugh commented on a diff in pull request #932: SOLR-15182: Remove remaining apispec files

2022-07-05 Thread GitBox
epugh commented on code in PR #932: URL: https://github.com/apache/solr/pull/932#discussion_r913890551 ## solr/core/src/java/org/apache/solr/handler/BlobHandler.java: ## @@ -22,6 +22,7 @@ import static org.apache.solr.common.params.CommonParams.SORT; import static org.apache.s

[GitHub] [solr] epugh commented on a diff in pull request #932: SOLR-15182: Remove remaining apispec files

2022-07-05 Thread GitBox
epugh commented on code in PR #932: URL: https://github.com/apache/solr/pull/932#discussion_r913888965 ## solr/core/src/java/org/apache/solr/api/ApiBag.java: ## @@ -58,6 +35,18 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import java.io.IOException; +import j

[GitHub] [solr] epugh commented on a diff in pull request #932: SOLR-15182: Remove remaining apispec files

2022-07-05 Thread GitBox
epugh commented on code in PR #932: URL: https://github.com/apache/solr/pull/932#discussion_r913888507 ## solr/core/src/java/org/apache/solr/api/ApiBag.java: ## @@ -17,37 +17,14 @@ package org.apache.solr.api; -import static org.apache.solr.client.solrj.SolrRequest.SUPPORTE

[GitHub] [solr] gerlowskija opened a new pull request, #932: SOLR-15182: Remove remaining apispec files

2022-07-05 Thread GitBox
gerlowskija opened a new pull request, #932: URL: https://github.com/apache/solr/pull/932 # Description Solr is in the process of moving away from 'apispec' defined v2 APIs, and towards annotated-based APIs. Only a few apispec files remain. # Solution This commit remov

[GitHub] [solr-operator] HoustonPutman merged pull request #449: Upgrade Zookeeper Operator dep to 0.2.14

2022-07-05 Thread GitBox
HoustonPutman merged PR #449: URL: https://github.com/apache/solr-operator/pull/449 -- 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 #443: Upgrade the Zookeeper Operator dependency to 0.2.14

2022-07-05 Thread GitBox
HoustonPutman closed issue #443: Upgrade the Zookeeper Operator dependency to 0.2.14 URL: https://github.com/apache/solr-operator/issues/443 -- 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 speci

[GitHub] [solr] epugh commented on pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on PR #924: URL: https://github.com/apache/solr/pull/924#issuecomment-1175010725 Thanks @cpoerschke for the reivew, I think you are most of the way but not all the way through right??? I *think* I've respnded toall of your feedback items. -- This is an automated message

[GitHub] [solr] epugh commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913746653 ## solr/core/src/test/org/apache/solr/update/processor/UUIDUpdateProcessorFallbackTest.java: ## @@ -155,17 +155,17 @@ SolrInputDocument doc(SolrInputField... fields) { }

[GitHub] [solr] epugh commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913729702 ## solr/core/src/test/org/apache/solr/update/processor/UUIDUpdateProcessorFallbackTest.java: ## @@ -155,17 +155,17 @@ SolrInputDocument doc(SolrInputField... fields) { }

[GitHub] [solr] epugh commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913714990 ## solr/core/src/test/org/apache/solr/handler/component/StatsComponentTest.java: ## @@ -1248,17 +1248,16 @@ public void testMiscQueryStats() throws Exception { // force

[GitHub] [solr] epugh commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913712611 ## solr/core/src/test/org/apache/solr/schema/TestCloudSchemaless.java: ## @@ -63,7 +63,8 @@ protected String getCloudSolrConfig() { @Override public SortedMap getExtraS

[GitHub] [solr] epugh commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
epugh commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913697233 ## solr/core/src/test/org/apache/solr/cloud/PeerSyncReplicationTest.java: ## @@ -289,15 +288,13 @@ private CloudJettyRunner forceNodeFailureAndDoPeerSync(boolean disableFinger

[jira] [Commented] (SOLR-16275) Replace Charset.forName("UTF-8") with StandardCharsets.UTF_8

2022-07-05 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562610#comment-17562610 ] Eric Pugh commented on SOLR-16275: -- I made the change globablly where it makes sense.

[GitHub] [solr] epugh merged pull request #930: SOLR-16281: review boxing unboxing

2022-07-05 Thread GitBox
epugh merged PR #930: URL: https://github.com/apache/solr/pull/930 -- 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] cpoerschke commented on a diff in pull request #924: SOLR-16276: redundant variable in tests

2022-07-05 Thread GitBox
cpoerschke commented on code in PR #924: URL: https://github.com/apache/solr/pull/924#discussion_r913655672 ## solr/core/src/test/org/apache/solr/handler/TestReplicationHandler.java: ## @@ -87,6 +87,7 @@ public class TestReplicationHandler extends SolrTestCaseJ4 { private

[jira] [Commented] (SOLR-16275) Replace Charset.forName("UTF-8") with StandardCharsets.UTF_8

2022-07-05 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562602#comment-17562602 ] Eric Pugh commented on SOLR-16275: -- Thanks for your comment [~mdrob] Should I co

[jira] [Resolved] (SOLR-16281) Review boxing/unboxing opportunties

2022-07-05 Thread Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16281?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Pugh resolved SOLR-16281. -- Fix Version/s: 9.1 main (10.0) Resolution: Fixed > Review boxing/unboxing opp

[jira] [Commented] (SOLR-16281) Review boxing/unboxing opportunties

2022-07-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562601#comment-17562601 ] ASF subversion and git services commented on SOLR-16281: Commit 1

[jira] [Commented] (SOLR-16281) Review boxing/unboxing opportunties

2022-07-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562600#comment-17562600 ] ASF subversion and git services commented on SOLR-16281: Commit 4

[GitHub] [solr-sandbox] markrmiller merged pull request #25: Cleanup some of the testing issues.

2022-07-05 Thread GitBox
markrmiller merged PR #25: URL: https://github.com/apache/solr-sandbox/pull/25 -- 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.a

[GitHub] [solr] ijioio opened a new pull request, #931: SOLR-16282: Update core admin handler to use pluggable custom actions

2022-07-05 Thread GitBox
ijioio opened a new pull request, #931: URL: https://github.com/apache/solr/pull/931 https://issues.apache.org/jira/browse/SOLR-16282 # Description At the moment implementing custom actions for `CoreAdminHandler` implies subsclassing of the `CoreAdminHandler` and overridng `han

[jira] [Comment Edited] (SOLR-16221) Solr Jenkins CI - s390x support

2022-07-05 Thread Nayana Thorat (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562400#comment-17562400 ] Nayana Thorat edited comment on SOLR-16221 at 7/5/22 7:05 AM: -

[jira] [Commented] (SOLR-16221) Solr Jenkins CI - s390x support

2022-07-05 Thread Nayana Thorat (Jira)
[ https://issues.apache.org/jira/browse/SOLR-16221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17562400#comment-17562400 ] Nayana Thorat commented on SOLR-16221: -- [~uschindler] For s390x nodes, puppet is not