dsmiley commented on pull request #225:
URL: https://github.com/apache/solr/pull/225#issuecomment-883026028
Thanks for this
--
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.
markrmiller commented on pull request #214:
URL: https://github.com/apache/solr/pull/214#issuecomment-882730941
bq. Otherwise, it seems a new module would be more appropriate.
You don't back up that statement with any support, other than:
bq. test-framework is about publishing
sonatype-lift[bot] commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672741219
##
File path: solr/benchmark/build.gradle
##
@@ -0,0 +1,140 @@
+/*
Review comment:
*Moderate OSS Vulnerability:*
### pkg:maven/com.goo
HoustonPutman closed issue #285:
URL: https://github.com/apache/solr-operator/issues/285
--
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-un
madrob commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r672783290
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
S
alittlec commented on issue #53:
URL: https://github.com/apache/solr-operator/issues/53#issuecomment-882472454
hi - is there any update on this?
--
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 t
madrob commented on pull request #216:
URL: https://github.com/apache/solr/pull/216#issuecomment-882803411
I tried to mess around with this a bit by creating a wrapping class, but
couldn't get it to work very cleanly so let's stick with this approach and
maybe can revisit in the future. Th
dsmiley commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672775627
##
File path: solr/test-framework/src/java/org/apache/solr/util/RandomizeSSL.java
##
@@ -104,10 +105,10 @@ public SSLRandomizer(double ssl, double clientAuth,
HoustonPutman merged pull request #288:
URL: https://github.com/apache/solr-operator/pull/288
--
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: issu
HoustonPutman closed issue #286:
URL: https://github.com/apache/solr-operator/issues/286
--
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-un
madrob commented on pull request #221:
URL: https://github.com/apache/solr/pull/221#issuecomment-882802608
> if getCurrentSchemaModificationVersion throws an exception, then its
caller doesn't have to catch the exception;
Is this an _is_ or _ought_ statement? In other words, are you
markrmiller commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672482454
##
File path: solr/test-framework/src/java/org/apache/solr/util/RandomizeSSL.java
##
@@ -104,10 +105,10 @@ public SSLRandomizer(double ssl, double clientAu
HoustonPutman merged pull request #287:
URL: https://github.com/apache/solr-operator/pull/287
--
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: issu
dsmiley commented on pull request #221:
URL: https://github.com/apache/solr/pull/221#issuecomment-883013519
> Is this an is or ought statement?
"ought". It's a minor nuisance really.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
dsmiley commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r672780636
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
Irfan Gazi created SOLR-15553:
-
Summary: How to create a Report
Key: SOLR-15553
URL: https://issues.apache.org/jira/browse/SOLR-15553
Project: Solr
Issue Type: Improvement
Security Level: P
[
https://issues.apache.org/jira/browse/SOLR-15553?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Atri Sharma resolved SOLR-15553.
Resolution: Not A Bug
Please use users mailing list for questions.
> How to create a Report
>
Irfan1 created SOLR-15554:
-
Summary: How to create a Report
Key: SOLR-15554
URL: https://issues.apache.org/jira/browse/SOLR-15554
Project: Solr
Issue Type: Improvement
Security Level: Public (D
dsmiley commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r672780636
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
dsmiley commented on pull request #225:
URL: https://github.com/apache/solr/pull/225#issuecomment-883026028
Thanks for this
--
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.
markrmiller commented on pull request #214:
URL: https://github.com/apache/solr/pull/214#issuecomment-882730941
bq. Otherwise, it seems a new module would be more appropriate.
You don't back up that statement with any support, other than:
bq. test-framework is about publishing
sonatype-lift[bot] commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672741219
##
File path: solr/benchmark/build.gradle
##
@@ -0,0 +1,140 @@
+/*
Review comment:
*Moderate OSS Vulnerability:*
### pkg:maven/com.goo
madrob commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r672783290
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
S
alittlec commented on issue #53:
URL: https://github.com/apache/solr-operator/issues/53#issuecomment-882472454
hi - is there any update on this?
--
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 t
dsmiley commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672775627
##
File path: solr/test-framework/src/java/org/apache/solr/util/RandomizeSSL.java
##
@@ -104,10 +105,10 @@ public SSLRandomizer(double ssl, double clientAuth,
HoustonPutman merged pull request #288:
URL: https://github.com/apache/solr-operator/pull/288
--
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: issu
HoustonPutman closed issue #285:
URL: https://github.com/apache/solr-operator/issues/285
--
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-un
HoustonPutman closed issue #286:
URL: https://github.com/apache/solr-operator/issues/286
--
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-un
madrob commented on pull request #221:
URL: https://github.com/apache/solr/pull/221#issuecomment-882802608
> if getCurrentSchemaModificationVersion throws an exception, then its
caller doesn't have to catch the exception;
Is this an _is_ or _ought_ statement? In other words, are you
[
https://issues.apache.org/jira/browse/SOLR-15554?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Cassandra Targett resolved SOLR-15554.
--
Resolution: Not A Problem
Again, please use the user mailing list to ask these question
HoustonPutman merged pull request #287:
URL: https://github.com/apache/solr-operator/pull/287
--
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: issu
madrob commented on pull request #216:
URL: https://github.com/apache/solr/pull/216#issuecomment-882803411
I tried to mess around with this a bit by creating a wrapping class, but
couldn't get it to work very cleanly so let's stick with this approach and
maybe can revisit in the future. Th
markrmiller commented on a change in pull request #214:
URL: https://github.com/apache/solr/pull/214#discussion_r672482454
##
File path: solr/test-framework/src/java/org/apache/solr/util/RandomizeSSL.java
##
@@ -104,10 +105,10 @@ public SSLRandomizer(double ssl, double clientAu
dsmiley commented on pull request #221:
URL: https://github.com/apache/solr/pull/221#issuecomment-883013519
> Is this an is or ought statement?
"ought". It's a minor nuisance really.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384270#comment-17384270
]
Mark Robert Miller commented on SOLR-12386:
---
Zookeeper coordinates. It’s one of
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384274#comment-17384274
]
Mark Robert Miller commented on SOLR-12386:
---
Who cares? Not many, I’ll agree wi
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384274#comment-17384274
]
Mark Robert Miller edited comment on SOLR-12386 at 7/20/21, 1:52 PM:
--
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384274#comment-17384274
]
Mark Robert Miller edited comment on SOLR-12386 at 7/20/21, 1:53 PM:
--
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384274#comment-17384274
]
Mark Robert Miller edited comment on SOLR-12386 at 7/20/21, 1:54 PM:
--
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384283#comment-17384283
]
Mark Robert Miller commented on SOLR-12386:
---
That’s actually the story of SolrC
marcosfpr opened a new pull request #228:
URL: https://github.com/apache/solr/pull/228
…objects
https://issues.apache.org/jira/browse/SOLR-15496
# Description
A static instance of the comparator(s) could be created and reused.
# Solution
I just
[
https://issues.apache.org/jira/browse/SOLR-15496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384315#comment-17384315
]
Marcos Felipe Pontes Rezende commented on SOLR-15496:
-
Hello everyone
[
https://issues.apache.org/jira/browse/SOLR-15496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384315#comment-17384315
]
Marcos Felipe Pontes Rezende edited comment on SOLR-15496 at 7/20/21, 2:58 PM:
chaicesan opened a new issue #290:
URL: https://github.com/apache/solr-operator/issues/290
As it stands we can't configure the lower level zookeeper cluster properties
(e.g. zoo.cfg) via the solr-operator. I realised this recently when I was
debugging one of our services and decided it's w
HoustonPutman commented on issue #264:
URL: https://github.com/apache/solr-operator/issues/264#issuecomment-883503503
Good to know.
So in the linked PR I added the ability to create the necessary
ServiceAccount when deploying the `solr` helm chart, but the operator itself
does not c
HoustonPutman closed issue #264:
URL: https://github.com/apache/solr-operator/issues/264
--
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-un
HoustonPutman merged pull request #283:
URL: https://github.com/apache/solr-operator/pull/283
--
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: issu
cpoerschke commented on a change in pull request #228:
URL: https://github.com/apache/solr/pull/228#discussion_r673272242
##
File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/LTRRescorer.java
##
@@ -53,6 +53,21 @@ public LTRRescorer(LTRScoringQuery scoringQuery) {
t
madrob commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r673288008
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
S
HoustonPutman closed issue #259:
URL: https://github.com/apache/solr-operator/issues/259
--
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-un
HoustonPutman merged pull request #284:
URL: https://github.com/apache/solr-operator/pull/284
--
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: issu
[
https://issues.apache.org/jira/browse/SOLR-15549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384398#comment-17384398
]
ASF subversion and git services commented on SOLR-15549:
Commit 9
cpoerschke commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r673293571
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -148,6 +148,8 @@ public static ExecutorService
newMDCAwareCachedThrea
[
https://issues.apache.org/jira/browse/SOLR-12386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384401#comment-17384401
]
David Smiley commented on SOLR-12386:
-
To be clear, my callous "who cares" was only a
HoustonPutman commented on pull request #120:
URL: https://github.com/apache/solr/pull/120#issuecomment-883573183
Been testing this out some more, and am finding some issues:
- Using the async API (not sure about synchronous), I'm seeing `status: 0`
when the backup actually fails. Th
athrog commented on pull request #120:
URL: https://github.com/apache/solr/pull/120#issuecomment-883590462
Regarding minio, I saw this too in my testing (I tried to make it work
before moving to S3Mock). Their handling of "directory-like" structure is
pretty incompatible with this project.
athrog commented on pull request #120:
URL: https://github.com/apache/solr/pull/120#issuecomment-883594327
I'll take a look at the async API issue you described. Most of my testing
lately (in the Solr 9 codebase) has been with the synchronous APIs, so that's a
good callout.
Regardin
marcosfpr commented on a change in pull request #228:
URL: https://github.com/apache/solr/pull/228#discussion_r673372216
##
File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/LTRRescorer.java
##
@@ -53,6 +53,21 @@ public LTRRescorer(LTRScoringQuery scoringQuery) {
th
marcosfpr commented on a change in pull request #228:
URL: https://github.com/apache/solr/pull/228#discussion_r673372216
##
File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/LTRRescorer.java
##
@@ -53,6 +53,21 @@ public LTRRescorer(LTRScoringQuery scoringQuery) {
th
marcosfpr commented on a change in pull request #228:
URL: https://github.com/apache/solr/pull/228#discussion_r673383681
##
File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/LTRRescorer.java
##
@@ -131,31 +146,12 @@ public TopDocs rescore(IndexSearcher searcher, TopDocs
sonatype-lift[bot] commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r673387063
##
File path:
solr/solrj/src/java/org/apache/solr/common/util/ObjectReleaseTracker.java
##
@@ -57,49 +55,55 @@ public static void clear() {
* @
HoustonPutman commented on pull request #120:
URL: https://github.com/apache/solr/pull/120#issuecomment-883620391
> Regarding minio
Yeah, I changed a few things and got it closer to working, but I guess that
problem was the last and unsolvable hurdle...
> Regarding the AwsReg
thelabdude opened a new issue #291:
URL: https://github.com/apache/solr-operator/issues/291
One emerging pattern in the K8s ecosystem I've noticed is to have a
cert-manager extension that issues a unique TLS certificate for each pod.
Typically how this works is the cert-manager extension w
athrog commented on pull request #120:
URL: https://github.com/apache/solr/pull/120#issuecomment-883624248
Both good points, thanks.
--
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
thelabdude opened a new pull request #292:
URL: https://github.com/apache/solr-operator/pull/292
Fixes #291
--
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
anshumg commented on a change in pull request #224:
URL: https://github.com/apache/solr/pull/224#discussion_r673407575
##
File path: solr/solr-ref-guide/src/major-changes-in-solr-9.adoc
##
@@ -170,6 +170,11 @@ Upgrade is to be done using Collection API
MIGRATESTATEFORMAT actio
HoustonPutman merged pull request #224:
URL: https://github.com/apache/solr/pull/224
--
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-unsubs
[
https://issues.apache.org/jira/browse/SOLR-15549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384450#comment-17384450
]
ASF subversion and git services commented on SOLR-15549:
Commit 1
[
https://issues.apache.org/jira/browse/SOLR-15549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Houston Putman resolved SOLR-15549.
---
Fix Version/s: 8.10
Resolution: Fixed
SolrJ 8.10 is now compatible with 9.x Clouds.
U
HoustonPutman opened a new pull request #293:
URL: https://github.com/apache/solr-operator/pull/293
Resolves #268
Still needs documentation.
But this gives the user the ability to set a TLS secret to do TLS
termination at the ingress for the common-endpoint only.
--
This i
madrob commented on a change in pull request #227:
URL: https://github.com/apache/solr/pull/227#discussion_r673477241
##
File path: solr/solrj/src/java/org/apache/solr/common/util/ExecutorUtil.java
##
@@ -196,7 +198,13 @@ public void execute(final Runnable command) {
S
[
https://issues.apache.org/jira/browse/SOLR-15254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17384517#comment-17384517
]
Mark Robert Miller commented on SOLR-15254:
---
I've got that workaround I think.
72 matches
Mail list logo